Doc : keras.utils.plot_model prints the shape as (None, n) but outdated (?, n) is given in doc

URL(s) with the issue:
https://www.tensorflow.org/guide/keras/functional
Description of issue (what needs changing):
keras.utils.plot_model prints the shape as (None, n) but given in doc as (?, n)
Clear description
Watch the line keras.utils.plot_model(model, "my_first_model_with_shape_info.png", show_shapes=True) in the doc link provided above.The output shown there is outdated
Correct links
Gist here to show the changes.
Tensorflow version: 2.4
Submit a pull request?
Yes

A PR has to be merged with keras team but it was mentioned in the tensorflow documentation.so i opened it here.

A PR has to be merged with keras team but it was mentioned in the tensorflow documentation.so i opened it here.

Hello! I am new to the open source community and would like to contribute to this issue.

Hello! I am new to the open source community and would like to contribute to this issue.

This is now fixed: https://www.tensorflow.org/guide/keras/functional

This is now fixed: https://www.tensorflow.org/guide/keras/functional


What is InAppBillingService doing in the Banner SDK?

We build Unity games with several plugins.
I'm wondering why the InAppBillingService is bundled as part of GoogleAds SDKs?

The Google Mobile Ads SDK supports in app purchase ads. v2.3.0 of the Unity plugin added support for this feature. This feature depends on the InAppBillingService. To make it easier to use the plugin, we include the InAppBillingService as part of the plugin.
If you 1) Aren't using in-app purchase ads, or 2) have other Unity plugins that already include this service, simply don't include the in-app-billing-service-aidl.jar file when importing the 2.3.1 release of the .unitypackage.


add zoxide feature

This PR adds a feature to install zoxide (https://github.com/ajeetdsouza/zoxide).
Tests have been added and they pass locally.

This project is not associated with the devcontainer project and appears to have been abandoned https://github.com/devcontainers-contrib/features/issues/628 . The official project has removed all devcontainer-contrib features from their listing. https://github.com/devcontainers/devcontainers.github.io/issues/451. There is a fork that is being worked on to update and maintain the features https://github.com/devcontainers-extra/features


Use async/await, ReceiveXml in sample?

Merely suggestions, feel free to overrule. I think it's valuable to demonstrate adherence to async/await pattern rather than using .Result. Also, ReceiveXmlFromJson seems like it would be for some fairly obscure cases, would it be better to demonstrate ReceiveXml here since it's more typical?

@tmenier Yep, all true. The ReceiveFromXmlJson is only in the test project so that I could use httpbin.org - not sure where to find something equivalent (and public) that would give me back the xml that I post.


Usage for configurations without disko

I've recently started to use nisos-anywhere to deploy my configurations which also inkludes a disko definition on machines that have a relative simple disk layout.
Now I'm struggling to use my data server as well as my Raspberry Pi with nixos-anywhere. The reason for the data server is that disko does not support a multi device setup for Btrfs at the moment so I would need to replace the disko step with a custom script. Same goes for the Raspberry Pi, since I'm using the UEFI Firmware on it the disk is already partitioned and only need to be mounted.
Would it be possible to have a flag like --skip-disko that requires/takes a script as argument which creates, formats and mounts the partitions to match the configuration given to nixos-anywhere? Basically a manual format and mount in the form of a script.
I know the use case is a bit niche but it would be helpful especially for the part with non supported setups by disko.

Just seeing this #69 , having a pre formated disk or a manual disk preparation via ssh would work with this. But having a disk configuration not supported by disko remains a problem.
I know that getting disko to work with this configuration would be the more desired solution, but it seems filesystems with multi device features (except zfs) remain a problem with disko.

For multi device btrfs, I'm using following workaround:
{
  lib,
  config,
  ...
}: {
  disko.devices = {
    disk.disk1 = {
      device = "/dev/disk/by-id/wwn-0x6000c2918383f7f5f16aa1866993b5d4";
      type = "disk";
      content = {
        type = "gpt";
        partitions = {
          boot = {
            priority = 1;
            size = "1M";
            type = "EF02";
          };
          root = {
            priority = 2;
            size = "100%";
          };
        };
      };
    };
    disk.disk2 = {
      device = "/dev/disk/by-id/wwn-0x6000c29543bc91b1bd5167d630966d24";
      type = "disk";
      content = {
        type = "gpt";
        partitions = {
          boot = {
            priority = 1;
            size = "1M";
            type = "EF02";
          };
          esp = {
            priority = 2;
            name = "ESP";
            size = "512M";
            type = "EF00";
            content = {
              type = "filesystem";
              format = "vfat";
              mountpoint = "/boot";
            };
          };
          root = {
            priority = 3;
            size = "100%";
            content = {
              type = "btrfs";
              extraArgs = ["/dev/disk/by-partlabel/disk-disk1-root" "-f" "-m raid1" "-d raid0"];
              mountpoint = "/mnt/root";
              mountOptions = ["compress=zstd"];
              subvolumes = {
                "/rootfs" = {
                  mountpoint = "/";
                  mountOptions = ["compress=zstd"];
                };
                "/home" = {
                  mountpoint = "/home";
                  mountOptions = ["compress=zstd"];
                };
                "/nix" = {
                  mountpoint = "/nix";
                  mountOptions = ["compress=zstd" "noatime"];
                };
                "/swap" = {
                  mountpoint = "/.swapvol";
                  mountOptions = ["compress=no" "noatime" "nodatacow"];
                  swap = {
                    swapfile.size = "2G";
                  };
                };
              };
            };
          };
        };
      };
    };
  };
}


[Request] Add Forge ITEM_HANDLER_CAPABILITY support

Peeking at the source code, it seems that the cables (and most other items from this mod) do not support the new Forge ITEM_HANDLER_CAPABILITY. Support for this should be added; otherwise, modders would be forced to use IInventory in order to have generic support for item pipes.

Looks like it has already been added in https://github.com/aidancbrady/Mekanism/commit/c12f1c1e6f521465ba7ea81c3ef0c39289ae115b.
Nice work @aidancbrady!

Will this be backported to 1.10?

Maybe a nice idea to add to this request: use the ItemCapability to interact with other mods. Currently all tileentities that interact with other blocks use IInventory (for example logistical pipes and the push of machines).
There amount of mods that only support capabilities is increasing. Would be nice to have a better cross mod interaction.

First part done by @aidancbrady at b97b7184c138000997e34194174a7d82e5ebbed4

Tested latest version (okey, version of 12 hours ago). Tested with Actually Additions crate as that only supports the inventory capability and not the IInventory.
Working:

Digital miner output to crate and vanilla chests
All processing machines automatic output to crate and vanilla chests
Pipes extracting from crate
Pipes inserting and extracting from chests

Not working:

Pipes inserting into crate

To test:

Something like RFTools inventory viewer (forgot the name) to test if Mekanism's capability is exposed correctly
Normal operations of non capability providing blocks (vanilla exposes both capability and IInventory)

Should be done!

Working flawless!!


Fix gcc 9 compile error when _POSIX_C_SOURCE=200809L is defined

CLA: trivial
Fixes #13049
Ensure that _GNU_SOURCE is defined to prevent compile errors with gcc 9 when _POSIX_C_SOURCE=200809L is defined

Thanks! I made now a new pull request (due to problems with amending the original commit) and I ensured that both appear as separate lines, but I still get the same error. Apologies if I'm overlooking something obvious. What did I do wrong this time? Thanks for your help.

CLA: trivial

Fixes #13049

Ensure that _GNU_SOURCE is defined to prevent compile errors with gcc 9 when _POSIX_C_SOURCE=200809L is defined

The problem is that the CLA tag is expected in the body of the commit message, not in the subject line. (There needs to be an empty line between subject line and body.) I took your last line, shortened it a little bit and made it the subject line. Also, I added some of your explanations from the GitHub thread to the commit message. The following result is just a suggestion, feel free to change at as you prefer.
 That was probably why you opened a new pull request, correct?

I hope I didn't scare you away with too much git technicalities? If you need help, please let me know.

I hope I didn't scare you away with too much git technicalities? If you need help, please let me know.

Thanks - you did all well.


External function for GeoTIFF output

Reported by Ansley B Manke on 12 Nov 2003 18:48 UTC
http://www.remotesensing.org/geotiff/geotiff.html
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/716

Comment by<EMAIL_ADDRESS>on 5 Nov 2004 19:05 UTC
changing "Severity" tag of bugs under Ferret Tasks to "enhancement"


Update sbt-scoverage to 1.7.2

Updates org.scoverage:sbt-scoverage from 1.6.1 to 1.7.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-scoverage" } ]

labels: sbt-plugin-update, semver-minor

Codecov Report

Merging #237 (f1d96be) into main (235bb66) will increase coverage by 0.43%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #237      +/-   ##
==========================================
+ Coverage   58.77%   59.21%   +0.43%
==========================================
  Files          30       30
  Lines         929      912      -17
  Branches       58       58
==========================================
- Hits          546      540       -6
+ Misses        383      372      -11

Impacted Files
Coverage Δ

...la/org/dhallj/imports/ReferentialSanityCheck.scala
90.00% <0.00%> (-1.67%)
:arrow_down:

...allj/tests/acceptance/AcceptanceFailureSuite.scala
87.50% <0.00%> (-1.39%)
:arrow_down:

...scala/org/dhallj/imports/CorsComplianceCheck.scala
92.30% <0.00%> (-1.03%)
:arrow_down:

...avagen/src/main/java/org/dhallj/javagen/Code.scala
0.00% <0.00%> (ø)

...ce/src/main/scala/org/dhallj/circe/Converter.scala
100.00% <0.00%> (ø)

...c/main/java/org/dhallj/javagen/ToCodeVisitor.scala
0.00% <0.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 235bb66...f1d96be. Read the comment docs.

Superseded by #239.


Update nokogiri to address CVE-2020-7595

https://github.com/advisories/GHSA-7553-jr98-vx47

this is superseded by #229


Check for errors before deferring body closure

On results from http, the body should be closed only if there was no
error. This patch moves all deferred closures after the error checks,
and adds a missing error check.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

* handles the trivial patch of the space missing in L191.

Yes, this was supposed to be left out! I just pushed an updated patch but Tom beat me by merging the patch a few seconds earlier.

Yes, this was supposed to be left out! I just pushed an updated patch but Tom beat me by merging the patch a few seconds earlier.

Sorry :)


Enhancement: Key ids for FieldArray items

I see an anti-pattern used in FieldArray: index as a key.
This could bring either to render glitches or to unnecessary re-renders.
My proposal is to wrap each array value in an array, like with es6 Map, e.g.:
[ uniqueKey, field ]

where the key is unique in the context of that particular FieldArray, i.e. an item added after one other is removed doesn't get the removed key (as it happens right now), but the next never used before.
The proposed use could be (taken from the example):
const renderMembers = ({ fields }) => (

    {fields.map(([index, member]) =>

        ...

    )}

)

Calling it index seems misleading. What if it were a unique id, e.g.
`fieldName[${++counter}]`

where counter was a "global" variable that was unique for every iteratee over the history of the web page?
I do agree that it needs to be differentiated from the second parameter, index, which is what you use to call remove(index). The array structure had not occurred to me, but with destructuring, it hardly matters.

Yesterday I was experimenting exactly on this idea, like:
fields.map(([key, member], index) => ...)

I think, although I don't know the project thoroughly yet, all the work could be done in createFieldArrayProps.js.

I'm not exactly sure which version could be more appropriate/understandable:
fields.map(([ key, member ], index) => ...) // 1

fields.map((member, key, index) => ...) // 2

fields.map(({ key, member }, index) => ...) // 3

The least convincing and unconventional is [2];
The most conventional (see Map) is [1], although I don't really like positional APIs, but that's just personal preference..

Probably not needed as we have fields.get(index) now

This issue was duplicated by #2735, closing as a duplicate of #2735.


error when installing composer require "intervention/image"  PHP5.5.9

I'm using PHP 5.5.9 that it has fileinfo enabled by default, and ubuntu 14.04.2, I even verified it with phpinfo.
And when I do
composer require intervention/image it gives the next error

Problem 1

intervention/image 2.2.1 requires ext-fileinfo * -> the requested PHP extension fileinfo is missing from your system.
intervention/image 2.2.0 requires ext-fileinfo * -> the requested PHP extension fileinfo is missing from your system.
Installation request for intervention/image ^2.2 -> satisfiable by intervention/image[2.2.0, 2.2.1].

Thanks

SOLVED:
it was not a bug of the intervention/image, I was running composer from another environment that it didn't has fileinfo configurated. It would be useful to see easily which php module extensions could recognize from composer.


errors when running "evaluate_methods.py"

I get the following warning and exception while running $ python evaluate_methods.py twitter
WARNING:
/usr/lib64/python2.7/site-packages/sklearn/metrics/classification.py:1074: UndefinedMetricWarning: F-score is ill-defined and being set to 0.0 in labels with no predicted samples.
'precision', 'predicted', average, warn_for)
EXCEPTION:
Traceback (most recent call last):
File "evaluate_methods.py", line 625, in
evaluate_twitter_methods()
File "evaluate_methods.py", line 461, in evaluate_twitter_methods
**DEFAULT_ARGUMENTS)
File "evaluate_methods.py", line 494, in run_method
return method(embeddings, positive_seeds, negative_seeds, **kwargs)
File "/home/michel/bin/socialsent/polarity_induction_methods.py", line 224, in bootstrap
polarities_list = pool.map(map_func, range(num_boots))
File "/usr/lib64/python2.7/multiprocessing/pool.py", line 251, in map
return self.map_async(func, iterable, chunksize).get()
File "/usr/lib64/python2.7/multiprocessing/pool.py", line 567, in get
Learning embedding transformation
raise self._value
Exception: Error when checking model target: expected y to have shape (None, 400) but got array with shape (36, 1)
Thank you for any feedback.

That warning is just an artifact of getting the F1-score when predicting the majority label baseline and is nothing to worry about. I am working on replicating your other error now.

Okay, it looks like an issue coming from keras and/or theano (so it should only happen when evaluating the "densify" method). Do you know what versions of these packages you have?

I've just installed all packages needed today, so it must be the latest versions.

Confirmed. Keras recently made some changes that are not compatible. You have two options: (1) install Keras 0.3.3 (should be possible via pip or manually from github)  or (2) remove/comment out all code that uses the "densify" method.
I will look into making the code compatible with the updated keras, but I unfortunately won't get to this in the immediate future. If you want to fix this and make a pull request that is welcome as well :)

Hi Will,
Back to you with some memory issues. My experience so far is that SocialSent runs into memory problem when you reach a threshold of more or less 7000 words to score. So I ran it on a distributed architecture (shartcnet) with 38000 words to score and ask for 16G memory, yet it very soon runs out of memory again:
...
Using Theano backend.
/opt/sharcnet/python/2.7.8/intel/lib/python2.7/site-packages/scipy/lib/_util.py:35: DeprecationWarning: Module scipy.linalg.blas.fblas is deprecated, use scipy.linalg.blas instead
DeprecationWarning)
Evaluating SentProp with 100 dimensional GloVe embeddings
Evaluating binary and continuous classification performance
LEXICON
SEEDS
EMBEDDINGS
EVAL_WORDS
Traceback (most recent call last):
File "concreteness.py", line 95, in
sym=True, arccos=True)
File "/home/genereum/socialsent-master/polarity_induction_methods.py", line 99, in random_walk
M = transition_matrix(embeddings, **kwargs)
File "/home/genereum/socialsent-master/graph_construction.py", line 62, in transition_matrix
return Dinv.dot(L).dot(Dinv)
MemoryError
--- SharcNET Job Epilogue ---
job id: 12138822
exit status: 1
cpu time: 313s / 12.0h (0 %)
elapsed time: 479s / 12.0h (1 %)
virtual memory: 11.9G / 16.0G (74 %)
Job returned with status 1.
WARNING: Job only used 1 % of its requested walltime.
WARNING: Job only used 0 % of its requested cpu time.
WARNING: Job only used 65 % of allocated cpu time.
WARNING: Job only used 74% of its requested memory.
...
A solution would be to run it 7000 words at time. But maybe you know a way to increase the memory use by the program?
Thanks,
Michel

Hi Michel,
I believe this is an issue with numpy. I've ran into these memory issues on a machine with 1Tb of RAM even when I was only using <100Gb. Basically, it is the Dinv.dot(L).dot(Dinv) that takes a lot of memory and you will need to distribute this matrix operation if you want things to be really efficient. Since the code only using the nearest neighbors from this computation, it could also be made more memory efficient by analyzing each row at a time and discarding useless info on the fly (if that makes sense), though this will have a big runtime penalty.
But I think numpy fails at these large matrix operations in general, so doing this right might be a bit tricky.  I was planning on addressing this at some point, but it definitely won't be for another couple weeks at least (I am traveling).
If you want to try to optimize Dinv.dot(L).dot(Dinv) and submit a pull request that would be great though!

Just to clarify: Numpy will fail to compute Dinv.dot(L).dot(Dinv) even when there is enough RAM due to internal memory size limitations. But the memory requirements are still huge in general and it will be necessary to make an optimized version of the Dinv.dot(L).dot(Dinv) computation.

I see. Let me ask you something else, if I may.
A colleague of mine wonders if we could improve performance by incorporating known predictors of concreteness, like word length/morphological complexity, word frequency, or some of the other semantic-space derived ones, like semantic diversity (log mean cosine distance between all a word's contexts). is there room in one of the polarity induction methods to include this? Or perhaps in the embeddings?
Thanks so much!
Michel

From: William L Hamilton<EMAIL_ADDRESS>Sent: July 4, 2016 6:57 PM
To: williamleif/socialsent
Cc: Genereux, Michel; Author
Subject: Re: [williamleif/socialsent] errors when running "evaluate_methods.py" (#1)
Just to clarify: Numpy will fail to compute Dinv.dot(L).dot(Dinv) even when there is enough RAM due to internal memory size limitations. But the memory requirements are still huge in general and it will be necessary to make an optimized version of the Dinv.dot(L).dot(Dinv) computation.
�
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/williamleif/socialsent/issues/1#issuecomment-230361798, or mute the threadhttps://github.com/notifications/unsubscribe/ATDnmjaH0ZFDf_02VrhwNz0WrGx1o2t1ks5qSY_hgaJpZM4I3x5f.

I think the most straightforward way to incorporate such features would be
to simply append them to the existing embeddings (and re-normalize the
embeddings after).
If you use the simple linear induction method then I think that this could
improve performance substantially (this method is also not very
memory-intensive). I am not as confident about how this would work the
other methods, however. My intuition is that this could work with the
SentProp-like methods (e.g., label_propagate_probabilistic), but that it
would not work as well with  densify, (since that method tries to find only
one embedding dimension that is most predictive of the label).
Let me know how it turns out!
Cheers,
Will
On Thu, 7 Jul 2016 at 06:38 genereum<EMAIL_ADDRESS>wrote:

I see. Let me ask you something else, if I may.
A colleague of mine wonders if we could improve performance by
incorporating known predictors of concreteness, like word
length/morphological complexity, word frequency, or some of the other
semantic-space derived ones, like semantic diversity (log mean cosine
distance between all a word's contexts). is there room in one of the
polarity induction methods to include this? Or perhaps in the embeddings?
Thanks so much!
Michel

From: William L Hamilton<EMAIL_ADDRESS>Sent: July 4, 2016 6:57 PM
To: williamleif/socialsent
Cc: Genereux, Michel; Author
Subject: Re: [williamleif/socialsent] errors when running
"evaluate_methods.py" (#1)
Just to clarify: Numpy will fail to compute Dinv.dot(L).dot(Dinv) even
when there is enough RAM due to internal memory size limitations. But the
memory requirements are still huge in general and it will be necessary to
make an optimized version of the Dinv.dot(L).dot(Dinv) computation.
�
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
https://github.com/williamleif/socialsent/issues/1#issuecomment-230361798>,
or mute the thread<
https://github.com/notifications/unsubscribe/ATDnmjaH0ZFDf_02VrhwNz0WrGx1o2t1ks5qSY_hgaJpZM4I3x5f

.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/williamleif/socialsent/issues/1#issuecomment-231079722,
or mute the thread
https://github.com/notifications/unsubscribe/AEO9DgjN0e8P_zUNsMkkmXFSb_t08GgEks5qTQE6gaJpZM4I3x5f
.

William,
Ok, to " append them to the existing embeddings (and re-normalize the embeddings after).", is there a particular python script in your package to start with or should I look toward something like word2vec ?
Cheers,
Michel

From: William L Hamilton<EMAIL_ADDRESS>Sent: July 12, 2016 2:01 PM
To: williamleif/socialsent
Cc: Genereux, Michel; Author
Subject: Re: [williamleif/socialsent] errors when running "evaluate_methods.py" (#1)
I think the most straightforward way to incorporate such features would be
to simply append them to the existing embeddings (and re-normalize the
embeddings after).
If you use the simple linear induction method then I think that this could
improve performance substantially (this method is also not very
memory-intensive). I am not as confident about how this would work the
other methods, however. My intuition is that this could work with the
SentProp-like methods (e.g., label_propagate_probabilistic), but that it
would not work as well with densify, (since that method tries to find only
one embedding dimension that is most predictive of the label).
Let me know how it turns out!
Cheers,
Will
On Thu, 7 Jul 2016 at 06:38 genereum<EMAIL_ADDRESS>wrote:

I see. Let me ask you something else, if I may.
A colleague of mine wonders if we could improve performance by
incorporating known predictors of concreteness, like word
length/morphological complexity, word frequency, or some of the other
semantic-space derived ones, like semantic diversity (log mean cosine
distance between all a word's contexts). is there room in one of the
polarity induction methods to include this? Or perhaps in the embeddings?
Thanks so much!
Michel

From: William L Hamilton<EMAIL_ADDRESS>Sent: July 4, 2016 6:57 PM
To: williamleif/socialsent
Cc: Genereux, Michel; Author
Subject: Re: [williamleif/socialsent] errors when running
"evaluate_methods.py" (#1)
Just to clarify: Numpy will fail to compute Dinv.dot(L).dot(Dinv) even
when there is enough RAM due to internal memory size limitations. But the
memory requirements are still huge in general and it will be necessary to
make an optimized version of the Dinv.dot(L).dot(Dinv) computation.
�
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
https://github.com/williamleif/socialsent/issues/1#issuecomment-230361798>,
or mute the thread<
https://github.com/notifications/unsubscribe/ATDnmjaH0ZFDf_02VrhwNz0WrGx1o2t1ks5qSY_hgaJpZM4I3x5f

.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/williamleif/socialsent/issues/1#issuecomment-231079722,
or mute the thread
https://github.com/notifications/unsubscribe/AEO9DgjN0e8P_zUNsMkkmXFSb_t08GgEks5qTQE6gaJpZM4I3x5f
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/williamleif/socialsent/issues/1#issuecomment-232128473, or mute the threadhttps://github.com/notifications/unsubscribe/ATDnmgmoeJLGuF7UJwAPAa0elcIBJtTnks5qU9Z9gaJpZM4I3x5f.


Is TextBlock weight useful with markdown?

It seems redundant, and I've noticed a few partners who are making cards just use markdown to bold the text block. They didn't even realize (or care) we had a weight property.

Fair point, closing


Line Follower Robot using Arduino #996

Have you read the Contributing Guidelines ?
(Write your answer here.)
YES
Description
(Write your answer here.)
The primary goal of this endeavor is to create an autonomous line follower robot, capable of tracking and following a predetermined path without the need for external guidance. It is capable of navigating a path that is marked with a prominent black line on a contrasting background.
Fixes #996

Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)

Hey @Kushal997-das
Can u please update me about me about this whether it's okay or do I need to change something?

@Kushal997-das please review the changes done.


Typo in translation

Here  the example in Czech language says: "this amount will be translated to Czech language and written on a cash order as Edno euro 01 cent." There is a typo - should be "Jedno", not "Edno" euro.
The same applies to THIS page.
Cheers, Lukas
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 031332a3-e979-c3fa-fb50-ef10857e7e25
Version Independent ID: c7d83f69-48e4-12df-84bb-d580befda0f4
Content: Update how amounts are displayed on reports and documents - Finance | Dynamics 365
Content Source: articles/finance/localizations/emea-amount-printing-forms.md
Service: dynamics-365-finance
Product: ****
Technology: ****
GitHub Login: @Anasyash
Microsoft Alias: kfend

@lukasnimberger, thank you for providing this feedback.
@kfend, could you please fix the typos.

Bug 647723 has been logged to track this issue.

The link provided in the comment and in the Document details section go to the same topic, Update how amounts are displayed on reports and documents. I have updated that topic and the changes will be published today. I am going to close this issue. If there is another topic with the same typo, please let me know which one it is. Thanks!


wher code

wher code

under there
On Wed, Jan 17, 2024, 4:04 PM hax4dazy @.***> wrote:

wher code
—
Reply to this email directly, view it on GitHub
https://github.com/Blessedseal/SpinFibsh.mp4/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWTJSOQOTKIWS6Z5BW5RJ5LYPBDIXAVCNFSM6AAAAABB7INZVGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA4DOMBXG43DSNA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Under where

haha! you said underwhere!1!
On Wed, Jan 17, 2024, 4:52 PM hax4dazy @.***> wrote:

Under where
—
Reply to this email directly, view it on GitHub
https://github.com/Blessedseal/SpinFibsh.mp4/issues/2#issuecomment-1897272210,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWTJSOTGYAZPUTY2FBE4WZDYPBI3NAVCNFSM6AAAAABB7INZVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJXGI3TEMRRGA
.
You are receiving this because you commented.Message ID:
@.***>

:trollface:

Same

i meant to send this to someone else my bad 💀
On Tue, Mar 5, 2024, 12:04 PM hax4dazy @.***> wrote:

Same
—
Reply to this email directly, view it on GitHub
https://github.com/Blessedseal/SpinFibsh.mp4/issues/2#issuecomment-1979343721,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWTJSOXJMYSK536SSALGPJDYWYCMXAVCNFSM6AAAAABB7INZVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZZGM2DGNZSGE
.
You are receiving this because you commented.Message ID:
@.***>

lmfao


Support customized environment variables for helm chart

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/Alluxio-PR-Builder/10344/
Test PASSed.

I will handle the related doc page change.

env:
# Extra environment variables for the master pod
# Example:
# JAVA_HOME: /opt/java

done

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/Alluxio-PR-Builder/10355/
Test PASSed.

alluxio-bot, merge this please.

@madanadit Do you think this should be cherry-picked into 2.3 branch? We are cherry-picking https://github.com/Alluxio/alluxio/pull/11786 with version 0.6.8, so I assume this 0.6.7 should be there too.

Yep, both

alluxio-bot, cherry-pick this to branch-2.3 please.

alluxio-bot, cherry-pick this to branch-2.3 please.


compile_commands.json does not use absolute paths for the compiler

This is a cross-post between meson and ninja.  It's not clear which tool should expand the compiler path to an absolute path. See discussion on Meson: https://github.com/mesonbuild/meson/issues/12834
Describe the bug
The generated compile_commands.json copies the name of the compiler as it can be found in the $PATH. According to the LLVM specification: "All paths specified in the command or file fields must be either absolute or relative to this directory". Because the command relies on $PATH, it is not possible to replay the compilation database if a different environment is used. I encountered this limitation when working with an external cross-compilation SDK. The meson workspace was configured inside the SDK environment and I'd like to replay the compilation in an IDE that doesn't source the SDK environment. I need absolute paths for this.
To Reproduce
I used the gtk-doc test-case as an example. I ran meson and ninja to produce compile_commands.json inside the build directory. The command field references the compiler without an absolute path:
[{ "directory": "/home/deribaucourt/Workspace/yocto-vscode/meson/test cases/frameworks/10 gtk-doc/builddir",
    "command": "cc -Ilibfoo.so.p -I. -I.. -Iinclude -I../include -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -g -fPIC -MD -MQ libfoo.so.p/foo.c.o -MF libfoo.so.p/foo.c.o.d -o libfoo.so.p/foo.c.o -c ../foo.c",
    "file": "../foo.c",
    "output": "libfoo.so.p/foo.c.o" }]

Expected behavior
compile_commands.json should contain the absolute compiler path instead of relying on $PATH to improve portability and respect the specification from LLVM:
"command": "cc => "command": "/usr/bin/cc
I had a look at the Meson and Ninja code myself to try to find a solution, but I need some pointers from the maintainers. My guess is that we need to resolve the compiler path before passing it to ninja. A less obtrusive way would be to only edit the .json file as a post processing phase.
system parameters

The issue is noticeable when cross-compiling mostly, but can be easily reproduced on native builds (by looking at the command field)
Ubuntu 22.04
Python 3.10.12
Meson 0.61.2
Ninja 1.10.1

This is very interesting, but I believe using paths relative to the directory where Ninja runs, or to the current PATH value is completely in-spec here.
Your use case is very specific, and should be served more easily by writing a custom tool to rewrite the content of the compile_commands.json for your IDE environment instead (by turning paths into absolulte ones, whenever possible).
Note that in general, this is not something that Ninja can handle reliably. For example, my project's build system injects multipe wrapper scripts before each compilation command, which look like this in the generated Ninja manifest:
rule compile_cxx
  command = path/to/wrapper1 <wrapper1_args> -- path/to/wrapper2 <wrapper2_args> -- <compilation command>

The point of each wrapper is to perform a different thing from the compilation command (e.g. wrapper2 sends the compilation command to a remote builder, while wrapper1 checks that output files do not include unexpected absolute paths that would break our shared caches. And here Ninja doesn't have enough information to translate some of the tool paths to absolute ones here.

Thanks for your answer,
Indeed, if I reconfigure my tool to invoke meson with an absolute path in meson.cross, then the output of ninja in compile_commands.json ends up with an absolute path as well. It's more appropriate than editing the .json after hand in my use case.
Your wrapper example also shows that ninja doesn't necessarily have the information to handle reproducing the initial environment and expects that the compile commands are invoked in the same one. It makes sense. I'll also keep that example in mind, I may have to use wrappers as well ;)

Oh, yes, having the generator provide absolute paths works well too.
For the record, certain generators, such as GN can also emit the compile_commands.json directly, and sometimes have extra configuration options to format its content.

I used clangd, which also doesn't find the desired compiler. So how do you let meson write absolute path to the compile_commands.json?

Thanks for your answer, Indeed, if I reconfigure my tool to invoke meson with an absolute path in meson.cross, then the output of ninja in compile_commands.json ends up with an absolute path as well. It's more appropriate than editing the .json after hand in my use case. Your wrapper example also shows that ninja doesn't necessarily have the information to handle reproducing the initial environment and expects that the compile commands are invoked in the same one. It makes sense. I'll also keep that example in mind, I may have to use wrappers as well ;)

I used clangd, which also doesn't find the desired compiler. So how do you let meson write absolute path to the compile_commands.json?

Hello @zwnewton, in my case I changed the environment variables providing C and CXX to contain the absolute path before invoking meson. You should try to edit whatever is providing relative commands to meson to provide absolute paths.


Logging improvement

fix: improve logging & refactor
This commit improves the logging, mainly adding "multi-field" values to
their own sub-field. Currently for example, all the resource info is
stored in top-level fields (group, version, name etc). For some
things this gets a bit hard to find, as there might be multiple
different names in a log line once it's been parsed. This commit
instead moves this resource-information into resource.<field> to group
those together.
Same goes for the HTTP error information.
refactor: improve metric handling
This commit improves the metric handling a bit, creating shorter locks.
refactor: extract helper functions
This commit extracts two useful helper function from the main Reconcile
loop, as that has grown over time and could have used a refactor.

:tada: This PR is included in version 0.15.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Default build settings at the app level

Describe the solution you'd like
I'd like to be able to specify a build configuration (or better yet a set of them) for an app as a whole, which can then be applied to branches of that app's codebase.  I'd then like to be able to choose to override, or not, individual build configuration options on a given branch.
Alternatively, if we could remove and re-clone a build configuration without losing the build history on that branch, that would help a lot (and I imagine may be easier for you all to implement?)
Describe alternatives you've considered
I don't think there are any alternatives that would 100% solve the problem (described below), but we've tried to mitigate difficulties by leaving as much build configuration as possible in our build scripts, by instituting policies on our team to not keep branches alive for any longer than necessary, and by just being very careful any time a global change has to be made to multiple build configurations.
Additional context
I'd hope you'd ask specifically for the problem I'd like solved, so I'll put that here.  ;)  The actual problem I'm having, which I think the above would solve, is that I work with a team of developers on multiple apps.   The devs will often start branches to implement new features, or to try fixing bugs, and may want to create builds in AppCenter to confirm that their changes didn't break the build.  This results in many branches, each with a copy of the build configuration.   Whenever we have to make a change to the build configuration, we often have to make those changes on most, if not all of our active branches to keep everything in sync.  Since we can't just remove and re-clone the build configuration without losing the entire build history on a branch, it becomes a lot of manual work to update every active branch individually, and we often find ourselves missing something.  Build configurations get out of sync.  We try to mitigate this by making sure to delete any branch as soon as it isn't needed, but that hasn't been enough.

+1
Another use case:
My current release pipeline design is based on feature branches off of master.  Once the PR passes a review, an app is built and distributed internally for verification of that specific branch.  In the case of binary distributions, branches in App Center need to be pre-configured as builds for ad-hoc branches cannot be programatically kicked-off and require manual steps to at least clone an existing configuration.
Alternatively (in my case) exposing through the api/cli the ability to clone configurations might be a good stop-gap.
I'm open to rethinking our automated pipeline if anyone wants to share how they are doing something similar or point to where this functionality might already exist.


TaxonomyEditor only displays "parts" taxonomy

Could be fixed by displaying a dialog to pick the taxonomy and then opening the palette on "Okay".
Could also use multiple BrowserCommandInputs and Tabs, but if using more than the three recommended taxonomies, this could get ugly fast in the future.

Fixed.


assisted-service-operator: crd updates

update to crds based on https://github.com/operator-framework/community-operators/pull/3978

/close
After talking with @zkayyali812


refactor params out, update template to support users array

better cut at #46  - comments added to init.pp, updated template

LOL - my foray into updating the RSPECs is meeting with resistance :)

I believe this is obsoleted with newer versions of the code.
I think params can still be factored out if you wish to do a new pr.


HADOOP-16186. NPE in ITestS3AFileSystemContract teardown in DynamoDBMetadataStore.lambda$listChildren

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
506
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
1054
trunk passed

+1
compile
30
trunk passed

+1
checkstyle
19
trunk passed

+1
mvnsite
35
trunk passed

+1
shadedclient
703
branch has no errors when building and testing our client artifacts.

+1
findbugs
40
trunk passed

+1
javadoc
21
trunk passed

_ Patch Compile Tests _

+1
mvninstall
29
the patch passed

+1
compile
27
the patch passed

+1
javac
27
the patch passed

+1
checkstyle
17
the patch passed

+1
mvnsite
31
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
718
patch has no errors when building and testing our client artifacts.

+1
findbugs
48
the patch passed

+1
javadoc
19
the patch passed

_ Other Tests _

+1
unit
276
hadoop-aws in the patch passed.

+1
asflicense
25
The patch does not generate ASF License warnings.

3672

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-628/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/628

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux f393073c2885 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 310ebf5

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-628/1/testReport/

Max. process+thread count
447 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-628/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

as usual: which S3 endpoint did you test with; which s3guard build options, etc...

refactored a little bit, and added a unit test

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
27
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1012
trunk passed

+1
compile
31
trunk passed

+1
checkstyle
22
trunk passed

+1
mvnsite
33
trunk passed

+1
shadedclient
691
branch has no errors when building and testing our client artifacts.

+1
findbugs
43
trunk passed

+1
javadoc
21
trunk passed

_ Patch Compile Tests _

+1
mvninstall
31
the patch passed

+1
compile
30
the patch passed

+1
javac
30
the patch passed

-0
checkstyle
16
hadoop-tools/hadoop-aws: The patch generated 1 new + 5 unchanged - 0 fixed = 6 total (was 5)

+1
mvnsite
34
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
752
patch has no errors when building and testing our client artifacts.

+1
findbugs
54
the patch passed

+1
javadoc
22
the patch passed

_ Other Tests _

+1
unit
275
hadoop-aws in the patch passed.

+1
asflicense
24
The patch does not generate ASF License warnings.

3180

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-628/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/628

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux 8340d03bc4e8 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 5257f50

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-628/2/artifact/out/diff-checkstyle-hadoop-tools_hadoop-aws.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-628/2/testReport/

Max. process+thread count
446 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-628/2/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

Code LGTM, only a couple of comments & it'll be good to go.
+1 pending those changes. thanks

Tested against ireland.
I still have one known issue: https://issues.apache.org/jira/browse/HADOOP-16207.
Maybe we can pull this, and fix that issue later?

[ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 66.011 s <<< FAILURE! - in org.apache.hadoop.fs.s3a.commit.staging.integration.ITestDirectoryCommitMRJob
[ERROR] testMRJob(org.apache.hadoop.fs.s3a.commit.staging.integration.ITestDirectoryCommitMRJob)  Time elapsed: 37.181 s  <<< ERROR!
java.io.FileNotFoundException: Path s3a://cloudera-dev-gabor-ireland/fork-0006/test/DELAY_LISTING_ME/testMRJob is recorded as deleted by S3Guard
at org.apache.hadoop.fs.s3a.S3AFileSystem.innerGetFileStatus(S3AFileSystem.java:2369)
at org.apache.hadoop.fs.s3a.S3AFileSystem.getFileStatus(S3AFileSystem.java:2343)
at org.apache.hadoop.fs.contract.ContractTestUtils.assertIsDirectory(ContractTestUtils.java:559)
at org.apache.hadoop.fs.contract.AbstractFSContractTestBase.assertIsDirectory(AbstractFSContractTestBase.java:327)
at org.apache.hadoop.fs.s3a.commit.AbstractITCommitMRJob.testMRJob(AbstractITCommitMRJob.java:133)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
at org.junit.rules.ExternalResource$1.evaluate(ExternalResource.java:48)
at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:298)
at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:292)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.lang.Thread.run(Thread.java:748)

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
75
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1390
trunk passed

+1
compile
39
trunk passed

+1
checkstyle
27
trunk passed

+1
mvnsite
43
trunk passed

+1
shadedclient
885
branch has no errors when building and testing our client artifacts.

+1
findbugs
60
trunk passed

+1
javadoc
30
trunk passed

_ Patch Compile Tests _

+1
mvninstall
36
the patch passed

+1
compile
36
the patch passed

+1
javac
36
the patch passed

+1
checkstyle
22
the patch passed

+1
mvnsite
39
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
953
patch has no errors when building and testing our client artifacts.

+1
findbugs
63
the patch passed

+1
javadoc
27
the patch passed

_ Other Tests _

+1
unit
283
hadoop-aws in the patch passed.

+1
asflicense
33
The patch does not generate ASF License warnings.

4125

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-628/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/628

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux 1f31ddf4858c 4.4.0-138-generic #164~14.04.1-Ubuntu SMP Fri Oct 5 08:56:16 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / a4cd75e

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-628/3/testReport/

Max. process+thread count
320 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-628/3/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

committed to trunk; closing


Local channels gone

There used to be epg for local channels now it is gone can someone fix this?

Please include the Chicago locals when you bring back locals (biggest tv market in central time zone).
WLS, WBBM, WMAQ WFLD

So far, only in test mode. If all goes well I'll add links to the guides to the list and let you know about it.

@TheDeatCrusher Local US and Canadian channels have been added to the list.
https://github.com/iptv-org/epg#us-states


Provide any information required for connecting to the MySQL Database (login credentials / settings)

Dependency for Issue #6, must be done early in the sprint to allow time for issue #9 to be completed.

This depends completely on how you install and setup MySQL on your computer. And is not needed.


Working but error showing after saving .vimrc

I have installed rainglow and all colorschemes working fine. But when i am saving vimrc, this error showing. I don't know why this error showing.

This error showing for few seconds only. then this error vanish.. Also colorscheme working fine.
By checking :colorscheme , it is showing rainglow.


Transfer & Loadout issues V3

A couple issues have come up trialing V3

I create a loadout with a specific set of armor. When i select the loadout, the chest peice and gloves have changed to a different pair. I have tried deleting the loadout and creating a new one, but the same issue occured.
In attempting to equip a set of gloves (both individually and in a loadout) i get the following error "Your character is not experienced enough to use this item... yet". The character i am transfering to is level 31 and should be capable of being equipped.

Otherwise this is looking to be an ace update. Keep up the good work.

Thanks for the description.  It's related to #63.


can not receive ipc message

i have a angular component that has the following code
this._electronService.ipcRenderer.send('ping', {'test': "test2"});
and in main.js i have
ipcMain.on('ping', (event, arg) => {
    console.log("print main");
    console.log(arg);
});

but the electron console always console.logs arg as undefined :(

@ThorstenHans can you help

it was a minor build mistake


drivers: flash: stm32 ospi flash driver the correct erase instruction

Use the most adapted instruction for the sector erase command It can be 0x20 or ox21 or 0x21DE in octo - DTR mode. The value is given by the SFDP table and  filled in the erase_types  table the during the SFDP discovery process.
This PR fixes an error found in the https://github.com/zephyrproject-rtos/zephyr/pull/66229

tested on stm32u585 disco kit with west build -p auto -b b_u585i_iot02a samples/drivers/spi_flash
--> sector erased  is correctly read at 0xFF
ospi-nor-flash@70000000 SPI flash testing
==========================

Perform test on single sector
Test 1: Flash erase
 erase cmd = 0x20
 erase cmd = 0x21de
 erase cmd = 0x21de
 erase cmd = 0x21de
Flash erase succeeded!
...

add another check in the samples/drivers/spi_flash after erasing,
In case the erase sector does not give 0xFF:
Perform test on single sector
Test 1: Flash erase
Erased Data do not match 0xffffffff

CI faiure due to https://github.com/zephyrproject-rtos/zephyr/issues/70445

@nashif CI is broken due to https://github.com/zephyrproject-rtos/zephyr/issues/70445 which is waiting for https://github.com/zephyrproject-rtos/zephyr/pull/70548. Would it be possible to merge it anyway ?
Current PR is fixing an issue seen by several users (see https://github.com/zephyrproject-rtos/zephyr/discussions/70844 for instance)

a CI failure remains (not related to this PR) when building
west build -p auto -b em_starterkit/emsk_em11d samples/drivers/spi_flash
even on SHA1  b9f3d68863b9f75dcc1dfac7dd7ff269b99cd118


Checkbox Point should be top aligned like bullets

Checkboxes in checklists should be aligned in the center of the first line, not the center of all the lines, similar to how bullets are positioned. With multiline checklists, it's really hard currently to tell where each line starts. iOS notes top aligns the checkboxes just like bullets to prevent them from moving around as the number of lines increases.

Also feel it.

I have the same issue, the alignment of the checkboxes feels off.

the same +1


fix(fedimintd): doesn't print version on start

Because it prints it before logging is initialized.

Backport failed for releases/v0.3, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin releases/v0.3
git worktree add -d .worktree/backport-5294-to-releases/v0.3 origin/releases/v0.3
cd .worktree/backport-5294-to-releases/v0.3
git switch --create backport-5294-to-releases/v0.3
git cherry-pick -x 6258ab83fd5425fa9275f502f397ebc61591351b


Remove the unused append-only arrow fragment.

What do these changes do?
The fragment is not unused anymore and we can stop maintianing it when we do some vineyard related refactors.

Codecov Report

Merging #1477 (f06e007) into main (daa8649) will decrease coverage by 38.14%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1477       +/-   ##
===========================================
- Coverage   73.60%   35.46%   -38.15%
===========================================
  Files          88       88
  Lines        9719     9719
===========================================
- Hits         7154     3447     -3707
- Misses       2565     6272     +3707

Impacted Files
Coverage Δ

...ython/graphscope/tests/unittest/test_cython_ast.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ython/graphscope/tests/unittest/test_serailaize.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...aphscope/tests/kubernetes/test_resource_builder.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/analytical/udf/patch.py
3.47% <0.00%> (-96.53%)
:arrow_down:

python/graphscope/tests/unittest/test_lazy.py
0.00% <0.00%> (-96.30%)
:arrow_down:

...thon/graphscope/tests/unittest/test_scalability.py
0.00% <0.00%> (-96.16%)
:arrow_down:

python/graphscope/tests/unittest/test_app.py
0.00% <0.00%> (-95.66%)
:arrow_down:

...hon/graphscope/tests/unittest/test_create_graph.py
0.00% <0.00%> (-92.00%)
:arrow_down:

...on/graphscope/tests/kubernetes/test_demo_script.py
0.00% <0.00%> (-91.75%)
:arrow_down:

python/graphscope/tests/unittest/test_graph.py
0.00% <0.00%> (-84.93%)
:arrow_down:

... and 47 more

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 daa8649...f06e007. Read the comment docs.


Use valid bucket end time instead of segment end time for merge/rollup delay metrics

Use valid bucket end time instead of segment end time for merge/rollup delay metrics. This handles the corner case that the metric take the segments which are not ready to merge into consideration.

Codecov Report

Merging #7827 (8950bb1) into master (90f5881) will decrease coverage by 57.05%.
The diff coverage is 91.66%.

@@              Coverage Diff              @@
##             master    #7827       +/-   ##
=============================================
- Coverage     71.66%   14.61%   -57.06%
+ Complexity     4089       80     -4009
=============================================
  Files          1579     1534       -45
  Lines         80832    78967     -1865
  Branches      12010    11807      -203
=============================================
- Hits          57932    11544    -46388
- Misses        19001    66571    +47570
+ Partials       3899      852     -3047

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
?

unittests2
14.61% <91.66%> (+<0.01%)
:arrow_up:

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

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


GEOMESA-1153 - Fix IN attribute queries without date component

Added unit test for queries without date in query

Signed-off-by: David Seapy<EMAIL_ADDRESS>

looks good

@jahhulbert-ccri Look ok to you?

looks good

fd07218021a3c11bc1ade3e1cf4a0445e8093405


Sc-15396 : invite teammate feature implementation

Scope of changes
This adds a few refactoring of access control and permission feature as well as invite teammmate feature
Fixes SC-XXXX
Type of change

[x] new feature
[ ] bug fix
[ ] documentation
[ ] testing
[ ] technical debt
[ ] other (describe)

Acceptance criteria
https://www.awesomescreenshot.com/video/16391141?key=381dd9fab59fe4468ab7504655c46148
Author checklist

[ ] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]  Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.
[ ] Are there any TODOs in this PR that should be turned into stories?

While testing this feature locally, I was unable to add a teammate. After clicking the save button, I was directed to the main page and when I returned to the teams page a new member wasn't listed. Here's a video: https://www.awesomescreenshot.com/video/16413212?key=411ed22d3058e85bd9b0181d33e10af0

@masskoder Is this due to the 401 error? I just merged a backend fix so maybe a merge from main will fix this?

@pdeziel @masskoder The fix that was recently merged solved the issue.


[pmp] Use top-level straps for PMP reset values

By using top-level straps for the PMP reset configuration its easier to implement different reset configurations if there are multiple Ibex cores in the system.

I had intentionally avoid doing this with parameters when I first wrote it to avoid adding yet more parameters to Ibex, though given we've already got loads perhaps it doesn't matter.
Ultimately this is probably what we have to do but I'll have a think about alternative methods.

@Razer6 we intend to take this, I just need to do a couple of checks to ensure there's no issues for non PMP configs or breakages in other things we don't check in CI I hope to get this done over the next couple of days.

@GregAC No hurry on that one. We have changed that in our environment and are not dependent on vendoring in a new version of ibex at the moment.

@GregAC @rswarbrick Could you take another look?

@Razer6 yes will do

Thanks @GregAC for the review. I just updated the changes for the nits.

Thanks @Razer6


No default output spaces in 20.0.0rc1

Up to now we've always had MNI152NLin2009cAsym as a default output space. Now I get
UserWarning: fMRIPrep will not generate preprocessed derivatives because none of `--output-spaces`, `--use-aroma`, or `--cifti-output` were set.

I would suggest we restore the default behavior.

This is the intended behavior, as the large refactor of output spaces makes it possible to add MNI152NLin2009cAsym normalization for the carpetplot generation and keep the workflow from breaking, but not clutter the output directory with (possibly) unwanted files.
And I believe it will still generate preprocessed anatomical? just not any func/ outputs.

The carpetplot bit was written for MNI152etc because that was the default output space, not the other way around. I'm happy to let users override the default space but having no BOLD outputs by default seems pretty extreme.

There needs to be a mechanism to allow users not to generate any outputs (although I don't really understand why you would do that).
One thing we could do is:

Roll back to the original behavior (i.e., if no --output-spaces is given, output MNI152NLin2009cAsym results).
Allow the user to write --output-spaces without any value, to indicate no bold conversions are to be generated.

WDYT?

That sounds fine.


WorldBank Datasets Refresh

download and process script modified and unit test script added

/gcbrun

/gcbrun

/gcbrun


add new university

gdgoenka university gurgaon

@ashishmangla0 Please provide us with:

the university official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the university
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the university recognizes the domain which you are submitting as an official email domain for the students.

@philipto I uploaded thinks you mentioned.

@philiptoI want to mention they have multiple domains like http://gdgoenka.ac.in/ only for emails. and for student/ teacher logins they are using "gdgu.org" only for emails.

@ashishmangla0 Please rename lib/domains/org/gdgu to lib/domains/org/gdgu.txt.
For the record: the screenshot above is not a proof, it just shows that a person who called themselves "Anjani K Goenka" registered gdgu.org domain with GoDaddy. It does not prove anything, because GoDaddy does not check an identity correctness of a person who pretends to be a domain registrant, and even if they checked it, a name of the person can not prove that the university officially recognizes the domain as the one registered for students and teachers.
Actual proof is, for example, here: https://www.ugc.ac.in/uni_contactinfo.aspx?id=692

@ashishmangla0 I am closing this request as already obsoleted. Thank you.


Row wise feature importance on scoring?

Hi how are you?
We are trying to better understand how features affect the scoring on a row by row basis. I believe it is theoretically possible in the prediction to generate X extra columns where X is the total number of features used by the model, and dial in for each row how many times each feature is used or if possible how much gain each feature brings to prediction.
If this is not done already, is it planned? Does it sound like a good idea? Could you give us a hint as to how to contribute it ourselves?
On the other hand... since there are so many trees and nodes for each row to go through, I suspect in the end the results won't vary much row by row, but I might be wrong and I'd like to see it myself.
As always thanks a lot!

If you are using R, "feature importance" per row is already available thanks to @yanyachen .

@ivangomezv any news?

Hey!
I can't work on this myself and it's not priority 1 so I'm hiring an external programmer to do it. We will definitely contribute it once it's done, I promise.

Hi,
I'm working on implementing this functionality in the Python package, but I'm finding inconsistencies between the data accessed by the R package in the file lgb.interprete.R and the data we're seeing in the Python package.
Specifically, when parsing the model with R, the program is looking for elements of the model that we're not finding when printing the model in Python:
single.tree.interprete <- function(tree_dt,
                                   tree_id,
                                   leaf_id) {

  # Match tree id
  single_tree_dt <- tree_dt[tree_index == tree_id, ]

  # Get leaves
  leaf_dt <- single_tree_dt[leaf_index == leaf_id, .(leaf_index, leaf_parent, leaf_value)]

  # Get nodes
  node_dt <- single_tree_dt[!is.na(split_index), .(split_index, split_feature, node_parent, internal_value)]

...

  # Perform leaf to root conversion
  leaf_to_root(leaf_dt[["leaf_parent"]], leaf_dt[["leaf_value"]])

  # Return formatted data.table
  data.table::data.table(Feature = feature_seq, Contribution = diff.default(value_seq))

}

When looking at the model in Python that we obtain from Booster.dump_model(), I can't see the elements leaf_parent or node_parent anywhere. Looking through the C++ Booster code, I can see that both R and Python end up referencing the same LGBM_BoosterDumpModel function (the R package calls a wrapper function, LGBM_BoosterDumpModel_R, that only seems to add some text encoding functionality).
Is there anything wrong in my approach? I'm trying to understand if there's any difference between the model data we're seeing R access and the model data we're manipulating with Python.

Sorry for replying late. You need to parse the boosting trees to a format that contain the parent index of a node or a leaf. I did that here: https://github.com/Microsoft/LightGBM/blob/master/R-package/R/lgb.model.dt.tree.R#L78

it seems this issue is solved. feel free to reopen if there are problems.


"comment" URL pre-fill param fails to work if comment contains equals sign (%3D)

Thanks for adding the URL parameter to pre-fill changeset comment (commit 21da76cf8060832e36, closing issue #2311). It works, but it seems to fail when there's an equals sign included in the comment (URL-encoded as %3D) ... which happens in quite a lot of HOT Tasks!
Here's the URL which I originally discovered the issue in, a URL constructed by the HOT Tasking Manager: http://www.openstreetmap.org/edit?editor=id&#map=13/-0.968384792013/20.7398745496&comment=DRC%2C+%23hotosm-ebola-drc-661%2C+%23MapGive%2C+source%3DWorldView-2%2C+DigitalGlobe%2C+NextView%2C+8+March+2011+or+16+September+2014&gpx=http://tasks.hotosm.org/project/661/task/48.gpx&background=custom:http://hiu-maps.net/hot/1.0.0/boende-2-flipped/{z}/{x}/{y}.png
The comment param is quite long. So I experimented to find a minimal reproducer and I found the following:

This works: http://www.openstreetmap.org/edit?editor=id#map=19/51.56473/-0.04403&comment=new changeset comment taken
This works: http://www.openstreetmap.org/edit?editor=id#map=19/51.56473/-0.04403&comment=new changeset comment%23taken (it has a URL-encoded hash sign in the comment)
This fails: http://www.openstreetmap.org/edit?editor=id#map=19/51.56473/-0.04403&comment=new changeset comment%3Dtaken (it has a URL-encoded equals sign in the comment)

Here, "works" means that if you make simple edit and press save, you see the desired comment pre-filled in the text box. "fails" means you just see the last comment you used in iD, whatever that was.
(BTW, I'm using Firefox v33)

The Firefox bug will be fixed in FF 41: https://www.fxsitecompat.com/en-US/docs/2015/urlutils-hash-no-longer-decodes-fragment/.

Browser fix has landed; this is now working in FF.


feat: customize k8s resources namespace based on extensions

What type of PR is this?
/kind feature

What this PR does / why we need it:
customize k8s resources namespace based on extensions
Which issue(s) this PR fixes:

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

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

13 of 31 (41.94%)  changed or added relevant lines in 1 file are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 54.307%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/engine/api/builders/appconfig_builder.go
13
31
41.94%

Files with Coverage Reduction
New Missed Lines
%

pkg/engine/api/destroy.go
7
82.35%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
8430

Relevant Lines:
15523

💛  - Coveralls


Platform configuration...

I wonder why there are 30 'ports' on the AHCI SATA Controller? I need about 4.
Why are there 35 'slots' in the Network Adapter section of the .VBOX file? I have only ever used 2.

Most probably not related to this script. The screenshot shows how a new VM is configured.

Thanks for the feedback:

Ports: this was the default value. Now reduced specifically.
Slots: seems to be a default value as well, however, seems no easy/obvious way to remove/limit them


Reboxing failures

The reboxing scheme described in section 10.1 of Compiling to categories sometimes fails. Collect examples here, and mull over improved schemes.

This example comes from compiling toEnum @Bool:
lam Case of boxer: bare unboxed var
  case n_ahgh of _ [Occ=Dead] { I# x_ahgk ->
  case x_ahgk of _ [Occ=Dead] {
    __DEFAULT -> $fEnumBool1;
    0# -> False;
    1# -> True
  }
  }

The issue arises from unlifted literal alternatives. Haskell does not have lifted literal alternatives except for Integer.
https://github.com/ghc/ghc/blob/master/compiler/coreSyn/CoreSyn.hs
Given that is the case, a reasonable solution would seem to be to convert the unlifted variables into lifted Integer and then convert each alternative to a literal Integer alternative. For many cases, this is trivial because the internal representation is also an Integer. For the case of Double and Float, they are represented as Rational, so it will be necessary to convert the Rational (two Integer) to a single Integer (probably using some bit shifting and or-ing them together).

I fixed case-of-Int# in 6af4d1a6aaa1117964e78710c011ce711d4ae6b8, with some improvements over the next few commits.


Fatal Error

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[42S02]: Base table or view not found: 1146 Table 'socialtr_fftcat.tcat_search_saved_archive' doesn't exist' in /home/xxx/public_html/ff-tcat/capture/query_manager.php:730 Stack trace: #0 /home/xxx/public_html/ff-tcat/capture/query_manager.php(730): PDOStatement->execute() #1 /home/xxx/public_html/ff-tcat/capture/search/savedArchive.php(20): getSavedBins() #2 {main} thrown in /home/xxx/public_html/ff-tcat/capture/query_manager.php on line 730
Any idea how to fix it?

Hi, @psegovias : This repository was forked from dmi-tcat for our research project. For some reason, we can't use chinese keywords in stream api. So, we add some tables in database to record our search bins, but we didn't put init sql statement in our code. Please execute the sql below, thx!
CREATE TABLE IF NOT EXISTS `tcat_search_saved_archive` (
`id` int(11) unsigned NOT NULL,
  `querybin_id` int(11) unsigned NOT NULL,
  `description` varchar(255) DEFAULT NULL,
  `origin_phrase` varchar(255) DEFAULT NULL,
  `createtime` datetime DEFAULT NULL,
  `updatetime` datetime NOT NULL,
  `savedtime` datetime DEFAULT NULL
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=1;

Nice! Thanks, now i get this:
"This capturing type is not defined in the config file"
When try to make a new bin capture.


pygmentize missing since 2.5

I don't see any mention of it being removed in the change notes. Is it supposed to be gone now?

It wasn't mentioned since it is still present in the installed package, just now as a setuptools console script.
For checkouts, use python -m pygments.

Ok, got it, thanks.

So does that mean this documentation needs to be updated to reflect you have to run it weirdly now?
https://pygments.org/docs/cmdline/

No, if you have installed it, there will be a pygmentize. This should only affect developers.


Copyright for BIDS specification is not stated anywhere

License without a copyright is like a driver license without a driver. Might not be the best analogy but the point is that there should be some "entity" holding the copyright while releasing works under some license.  Since there is no particular person or official entity, I would recommend to do the same as many other projects do - announce that copyright belongs to the project and/or contributors. Something like
Copyright: 2020, BIDS Contributors
Issue came up to properly attribute copyright and license to BIDS logo derivative: a perspective logo for Open Brain Consent project https://github.com/con/open-brain-consent/pull/69

Good point, PR welcome :-)
I agree with "BIDS Contributors" and I have also seen this custom over a wide array of projects.

+1 sounds good to me!

@yarikoptic do you want to state Copyright: 2020, BIDS Contributors at the top of the LICENSE file? or did you have another place in mind?


upgrade git-repository 0.30 to gix 0.36

This upgrade is mainly to benefit from the new names of all git-* crates which now share the gix- prefix, with gix naturally being what was formerly git-repository.
Now there is no need anymore to rename crates in order to get the git namespace, instead we refer to gitoxide as gix from the get-go.

Interestingly I thought exactly the same when doing this PR, just one more feature to go!
Unfortunately I have to follow priorities dictated by cargo for this half of the year which doesn't include this feature. Fortunately though, we don't have to wait that long as @pascalkuthe has already indicated interest in the task and I couldn't be more excited - helix is waiting for this feature too and I know we are going to get a high-performance, state-of-the-art implementation once he gives it a go 🎉.


Evaluation Script?

Thanks for releasing this work! I want to reproduce the WMDP results for a paper. What is the prompt format that was followed in the paper? It would be great if there was a script to evaluate an unlearned model on WMDP. Thanks!

Hello Kyle -- thanks for your interest! We've implemented WMDP in lm-evaluation-harness for standardized prompting. The template is borrowed from MMLU.

Very cool! Glad to hear WMDP is on the eval harness. :)


handling metrics correctly

This pr Closes #1787 .
Description:
Where I have considered the thershold to be CacheStrikes value to increment the metrics event_cache_process_info_failed && event_cache_parent_info_failed  and not on all retries . On retries we will inc the values
event_cache_parent_info_retry and event_cache_process_info_retry

One more note, you can ask Github to automatically close the issue after the PR is merged by adding e.g. Fixes #1787 to the PR description (see https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue for details).

Thanks for helping me to improve this @lambdanis . Please have  look at them

Also @willfindlay FYI, as you have context here you might have some more insights or suggestions.

Thank you for holding on @lambdanis . I hope the new changes match the expectations

Thank you @lambdanis

Thanks, can you also update the commit message to contain some description of the change and only one sign-off?

Hope its ready to merge

@olsajiri  actually that was done by gofumpt formatter . reverted those changes thanks 🙂

Thanks. Need to do a deeper review pass later but for now I left a nit for you.

sure , thanks @willfindlay


Etda 1827

ProQuest agreement additions

Waiting for Graduate School to give us the wording for the ProQuest statement.

Waiting for Graduate School to give us the wording for the ProQuest statement.


[Docs]: Modification required for the reCaptcha documentation under the "Validation Example" section

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
https://docs.appsmith.com/reference/widgets/button/google-recaptcha
Describe the problem
In the reCaptcha document under the "Validation Example" section it shows to paste the generated secret key values in the 'header'

Describe the improvement
In the reCaptcha document under the "Validation Example" section it shows to paste the generated secret key values  need to be sent in the body as post params i.e form-data and not in the headers.

Duplicate of #15863.


Limit number of user addresses

add migration to left only 100 addresses per user
introduce the settings that defined the maximum amount of addresses per user
do not save a new address in the checkout if the user already has max amount of addresses
when a new address to the user with a max number of addresses, remove the older one

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-5936 Users with a lot of addresses can load overload Saleor.


Link the concurrency runtime against libatomic on Linux

We mostly get away without this because we're fairly disciplined about using constant memory orderings, and apparently that's usually good enough to get inline accesses and avoid needing to link atomic.  However, we have a few places with the task status atomic that use a non-constant load ordering with load and compare_exchange_weak, and my recent change to make that atomic a double-word was apparently sufficient on some (but not all) Linux distributions to get the compiler to call the runtime function.  Regardless, we shouldn't be playing around in the margins here: Linux requires us to link libatomic, so we should.

Huh, thanks for the explanation in the opening comment, good to know!

We will need this change on release/5.5 branch too, it's also failing there.
https://ci.swift.org/view/Swift 5.5/job/oss-swift-5.5-package-linux-ubuntu-18_04/248/console


Installing dbt-coves downgraded my dbt installation (with specified adapter) to 0.18.2 [BUG]

Describe the bug
I had dbt-redshift installed in a pipenv, and when I installed dbt-coves it looks like it installed<EMAIL_ADDRESS>(without a specific dbt adapter specified)
To Reproduce
Steps to reproduce the behaviour:

Create a new pipenv
pipenv install dbt-redshift
Run dbt --version, note that plugins postgres and redshift are installed at 0.19.2
pipenv install dbt-coves
Run dbt --version, note that all 4 core plugins (the two above plus snowflake and BQ) are installed at 0.18.2
Run pipenv update dbt-redshift
Run dbt --version and get

installed version: 0.19.2
   latest version: 0.19.2

Up to date!

Plugins:
  - bigquery: 0.18.2
  - postgres: 0.19.2
  - redshift: 0.19.2
  - snowflake: 0.18.2

Note that BQ and SF are still 0.18.2 but PG and RS are 0.19.2

Expected behaviour
Installing dbt-coves shouldn't clobber an existing dbt install which has specified a specific adapter
Desktop (please complete the following information):
Windows 10
Python 3.8
dbt-coves v0.19.2-a.10

Hello @joellabes,
Could you please try with new dbt version 0.20.0 and then let us know?
In my case, It is now downgrading the version of BG and SF with new version. Also, before when I installed dbt-coves for dbt 0.19.2, It didn't downgrade the version.
However, I was using mac, with python 3.8, and dbt-coves v0.19.2-a.10.
Please let us know if it is creating the same issue with 0.20.0 in your machine.

Could not reproduce the bug.


ember exam expects ember binary on path

We're seeing command not found: ember when trying to run ember exam.
It looks like ember exam is expecting ember to be on PATH, but this may not always be the case. Any concerns with invoking ember from within node_modules?
https://github.com/trentmwillis/ember-exam/blob/67e400a842010219b5bab38f4b70b95ab05011fb/lib/commands/exam/iterate.js#L77

It looks like ember exam is expecting ember to be on PATH

node also resolves all the binaries located in the ${projectDir}/node_modules/.bin/.
ember-cli usually provides node_modules/.bin/ember binary. Do you have ember-cli in you package.json?


angles: 1.9.12-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository angles to 1.9.12-1:

upstream repository: https://github.com/ros/angles.git
release repository: https://github.com/ros-gbp/geometry_angles_utils-release.git
distro file: melodic/distribution.yaml
bloom version: 0.9.0
previous version for package: 1.9.11-0

angles
* Added support for "large limits" (#16 <https://github.com/ros/angles/issues/16>)
* Small documentation updates.
* Contributors: Franco Fusco, Tully Foote

hold until after next melodic sync


Implementing an Admin group within rancher that can be tied to an AD group

have a LDAP group map all users to the admin role within rancher.

Agree. The concept is quite confusing that the first user which logs into rancher after activating LDAP/AD auth provider is the admin user. This makes is especially difficult for automating the setup process.

Furthermore, it would be quite helpful for having an admin user beside the particular auth provider enabled. So even if this above mentioned AD/LDAP group resolves to no user, that you are able to manage your setup.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Allow response body of any size to be compressed

Hi @essen,
I've been looking for  a way to compress all responses, regardless of their size. I noticed that this is currently hardcoded in Cowboy 1 and 2.
In Cowboy 2 there is a comment to maybe make this configurable. I'd be happy to work my way around the codebase to make this happen if:
1/ that's still something you believe valuable
2/ implementation is not too difficult for a newcomer
Thanks
Cowboy version: 1.1.2
(Used via Phoenix 1.3.0)

Yes it's still valuable. This module is actually still experimental so this kind of feedback is very useful.
https://github.com/ninenines/cowboy/blob/master/src/cowboy_metrics_h.erl#L139 has an example of option value, it's just a matter of calling maps:get and keeping the option around in the state. Just make sure the default is what's currently in the code and add a test in compress_SUITE with a custom value and that should be good. You can run the compress test exclusively via make ct-compress.
I have no plans for further Cowboy 1 versions though.

Ok, thanks, I'll have a go at it.
No problem for Cowboy 1, having this in Cowboy 2 will be good already.

Merged with tweaks.


Prepare release v0.2.1

:sparkles: No lint errors found. :sparkles:

Coverage remained the same at 100.0% when pulling 83e910d6b756606bfe934e3afe4ba09d0f0a5e06 on version-bump into 328da0129c39225dffd33a9e267cf65e27951c8f on master.

👍


Add message info object

This change adds a context-local variable that exposes the message information in any message handling context.  The new context-local proxy, current_message, can be used as though it was the actual dbus_fast.message.Message object.  The context-local has its value set at the start of _process_message.
I wanted to discuss this change as it's one of the many ways that this information could be exposed to services; but the change is fairly self-contained and easy to understand so the best way of showing what I'm proposing the change is to just raise the PR.  Feedback welcome.

Apologies for getting my commit message formats wrong.  If someone can suggest how I can fix them on an open PR, I am more than willing to do so, but AFAICT it is either not possible or a Very Bad Idea to edit the commit messages of already-pushed commits.

I'm closing this PR since the other one has gone stale without a response. https://github.com/Bluetooth-Devices/dbus-fast/pull/170#issuecomment-1340161200


Villages causing crashes 1.10.2 forge

https://gist.github.com/mindcloud69/1dbce8f987da9aeddb7f73bee2db99dc

Yeah this is my problem too...

Was this on a server or a client version of Forge?

server

both

forge <IP_ADDRESS>5

Here is another Crash Report:
I restarted my world and everything booted up just fine. Running around without issues so far.
crash-2017-03-17_06.07.15-server.txt


docs/Maintainer-Guide: Fix BuildPulse links

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

https://github.com/Homebrew/brew/actions/runs/7596213836/job/20689665591?pr=16512

Thanks @issyl0!


Give user an opportunity to handle exceptions occuring in websocket read_data

This pull request resolves the problem of unhandled exceptions mentioned in #258 during reading data from websocket by giving user a way to handle exceptions. Now one can handle the exceptions in on_error callback e.g. firing a reconnect after a lost connection.

👍 Looks great, yes please, this functionality would be really useful

Please add this functionality. This is really important to detect connection errors.


[NNPACK BUG] MXPredGetOutput occasionally stucked when getting the 2nd output node

The bug only shows up when using NNPACK as backend.
When I am trying to MXPredGetOutput from multiple node, the result from 1st node would always be got successfully, however the 2nd node would have a low chance to stuck forever. It shows up occasionally, but it is critical for high frequency tasks such as face detection.
I also confirm that it happens on android device and in older version like v0.8
GDB shows this function is waiting for some threads.
Environment info
Operating System: Ubuntu 14.04 / android
Compiler: GCC
Package used:  NNPACK
MXNet version: v0.93 / v0.8
GDB debug message when paused:

0	pthread_cond_wait@@GLIBC_2.3.2		185	0x7ffff7bc8404
1	std::condition_variable::wait(std::unique_lockstd::mutex&)	/usr/lib/x86_64-linux-gnu/libstdc++.so.6		0x7ffff3c622bc
2	mxnet::engine::ThreadedEngine::WaitForVar(mxnet::engine::Var*)	/home/pallas/mxnet/lib/libmxnet.so		0x7ffff40af409
3	mxnet::NDArray::SyncCopyToCPU(void*, unsigned long) const	/home/pallas/mxnet/lib/libmxnet.so		0x7ffff412ba58
4	MXPredGetOutput	/home/pallas/mxnet/lib/libmxnet.so		0x7ffff40a42ae
5	GetOutput	mxnet_cpp_helper.cpp	186	0x4ed828
C++ source code
`//
/* 3 Do Predict Forward */
MXPredForward(pred_hnd);

// 4 Get Output Result
int output_size = 0;

/*bbox_pred, cls_prob*/
vector<mx_uint> output_index = {0, 1};
vector<mx_uint> bbox_shape;
std::vector<float> bbox_data;
GetOutput(pred_hnd, output_index[0], bbox_shape, bbox_data, output_size);

vector<mx_uint> cls_shape;
std::vector<float> cls_data;
GetOutput(pred_hnd, output_index[1], cls_shape, cls_data, output_size);

`
` //
void GetOutput(PredictorHandle pred_hnd, const int output_index, vector<mx_uint> &shapes, std::vector<float> &data, int &output_size){
mx_uint *shape = 0;
mx_uint shape_len;
shapes.clear();
// Get Output Result
MXPredGetOutputShape(pred_hnd, output_index, &shape, &shape_len);
size_t size = 1;
for (mx_uint i = 0; i < shape_len; ++i){
    size *= shape[i];
    shapes.push_back(shape[i]);
}
std::cout<<"shape: "; for (mx_uint i=0;i<shape_len;i++) std::cout<<", "<<shape[i]; std::cout<<std::endl;
/* prepare the space for output */
data.resize(size);
output_size = size;
MXPredGetOutput(pred_hnd, output_index, &(data[0]), size);
}

`

I recently fixed a race condition in pthreadpool (see Maratyszcza/pthreadpool@630dfb6638927df1f3181eee339662a86b3cea0b), the threading library used by NNPACK. This race condition sometimes resulted in deadlock. If you use an earlier pthreadpool version, this is the likely cause of your problem.

@Maratyszcza
fixed, love you.


Tree Matcher/reconstructor incorrectly (un-)flattens children

Describe the bug
Sometimes (e.g. not reliable/random chance), the TreeMatcher (and therefore the Reconstructor applies trees like this:
expr
    literal "a"
    "5"
    "5"

as if they were
expr
    expr
        literal "a"
        "5"
    "5"

To Reproduce
parser = Lark.open_from_package("lark", "lark.lark", ("grammars",))

lark_reconstructor = Reconstructor(parser, {
    "_NL": lambda _: "\n",
    "_VBAR": lambda _: "|",
})

pattern = parser.parse("""
start: "a"~5..5
start: ("b"~5)~5
""")
print(pattern.pretty())
print(lark_reconstructor.reconstruct(pattern))

This script sometimes wrongly produces
start:("a"~5)~5
start:("b"~5)~5

(I came across this while working on ast_generator)

Maybe related: Sometimes, the tree_matcher outputs normally illegal constructs of the shape Tree('expansion',[Tree('expansion',[...])]) or Tree('<origin>', [Tree('<alias>',[...])]) This can be dealt with, but is kinda annoying. I don't have a simple example. Here is where I deal with it.

Sounds like something that can be solved by changing priority / ordering of choices?

@erezsh I am not sure. I don't know exactly how tree_matcher is implemented.


Undocumented Feature: publishConfig: { @scope:registry: "" }

Publishing a scoped package is still somewhat undocumented in terms of the publishConfig.
Here is what I had to do in package.json to make it work with a Nexus instance:
{
  "name": "@scope/package",
  "version": "1.0.0",
   ....
  "publishConfig": {
    "@scope:registry": "https://nexus.host.com/repository/npm-private/"
  },

Then you need to use npm login --registry=https://nexus.host.com/repository/npm-private/ to log in. For the CI, I had to use https://www.npmjs.com/package/npm-cli-login package
Ref:

https://github.com/npm/www/issues/231
https://github.com/npm/cli/issues/6652

@MylesBorins where can I see the effect of this?


[ADP-3214] Remove DBFresh from usage sites

In preparation for the improvement of the database initialization logic, this pull requests removes all usage sites of the DBFresh type (except in Cardano.Wallet.DB.Layer).
Instead of bootDBLayer and loadDBLayer, use the corresponding functions

withBootDBLayerFromFile or newBootDBLayerInMemory
withLoadDBLayerFromFile

Issue number
ADP-3214

LGTM

Thanks. 😊

even if I've lost a bit of grip on the overall design now 🤷

The design will hopefully become more apparent later on:
The goal is that withBootDBLayerFromFile and withLoadDBLayerFromFile become the primary/only ways of obtaining a DBLayer from a database file path. The name of the function indicates whether the file is supposed to be initialized, or whether it already contains data (and needs migration). In both cases, the result value of type DBLayer is guaranteed to hold a valid WalletState.


Inset the full screen modal used for workspaces on large devices

cc @marcaaron
Details
Added the expected spacing around the full-screen modals without modifying too many things.
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/169135
Tests / QA Steps
Using a large screen (like the web version or the desktop app):

Create a new workspace
Go to its settings page
Verify that the full-screen modal is separated from the window border and that its corners are rounded, like this photo:

Using a small screen (mobile version):

Create a new workspace
Go to its settings page
Verify that the full-screen modal is not separated from the window border and has no rounded corners, like in this photo:

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

Should we set the overlay background color to the one used in the Figma design or leave it as rgba(0, 0, 0, 0.5)?

I think we should already have a color variable for the correct overlay color. If you launch an attachment modal from a chat, you should see what I mean.

Done! I had to create a new component to customize the background color. Let me know what you think about the changes.

I'm not sure if this is a blocker, but interacting with the backdrop won't dismiss this modal.

We could use a Touchable inside the CardOverlay to handle these interactions. However, the visible surface of the overlay is tiny (20px per side), so it may not add that much value.
Have we considered adding support for the Esc key to close modals? I think that would be more suited for this use case.

We could use a Touchable inside the CardOverlay to handle these interactions. However, the visible surface of the overlay is tiny (20px per side), so it may not add that much value.

I'll defer to @shawnborton on this one. I see your point about the small surface, but also can see an argument where the universal expected behavior for a backdrop behind a modal (however small) would be to dismiss the modal. No strong feelings on it though :)

Have we considered adding support for the Esc key to close modals? I think that would be more suited for this use case.

esc key should already be enabled, yes

I don't feel strongly about the outside area being clickable. I agree with Marc though that it is expected behavior, but I could live without it here. I feel like the Esc key is more useful though so if I had to pick, I would want the Esc key before the outer area.

I'll pick this back after coming back from OOO 😄

I just fixed the bug @marcaaron mentioned for the Reports screen. I still need to test other screens similar to the Reports one to see if there are similar regressions.

Let us know when it's ready for another review 👍

Ready for review!

I have read the CLA Document and I hereby sign the CLA

I'm not sure if this PR was blocked on me (sorry, if so) but it has conflicts now.

Conflicts fixed. If you could review @marcaaron

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @marcaaron in version: 1.0.83-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @isagoico in version: 1.0.85-9 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Online demo is down

Self-explanatory. Online demo is down.
https://whisperui.monsterapi.ai/

Thanks for flagging @samueldashadrach. I've just pinged the Monster API folks.


Use Whippet v2.3.0

@RobjS I think that job failed, so I'll re-run it once, in case it's an ephemeral problem

@snim2 I think the failure was from the draft PR before I'd updated the SHA for the download. I've pushed the amended commit now with the new SHA, so 🤞


Add USB tests

How to run these?
For the test_class example you need to clone usb-device, cd to it and run cargo test with the connected hardware running the test_class example. Or first observe dmesg, it shows errors right now.
For the serial example you need to clone serial-threadmark and run cargo run --release /dev/serial/by-id/usb-Fake_company_Serial_port_TEST-if00 1000000 a couple of times. None of the runs should fail.

these were added in https://github.com/nrf-rs/nrf-hal/pull/295, so closing this


changed style of ride page

The change in style of ride page is done .

made ride page more attractive


[Feature Request] TorchRL for Python 3.13t

Hello, PyTorch recently added support for Python 3.13t, aka "no-gil" Python (https://github.com/pytorch/pytorch/issues/130249).
Are there any plans to make TorchRL usable from this version? It would be nice if this could be supported soon, as the ability to run CPU code for interactions with the environment from multiple no-gil threads could lead to major performance improvements and higher GPU utilization.

That's definitely on our radar!
Beyond multithreaded envs another big benefit would be reducing the cpu overheads across the lib (sampling in rbs, iterating over TensorDict etc).
Is there anything we should prioritise in your opinion?

Is it somehow possible to install torchrl with a 3.13t python right now from nightly builds?


Add sync and suspend for image update automations

Allows sync and suspend for image update automations + fixes a bug where buttons were not disabled when un-checking the last checked button (having variable outside of useEffect prevented update -> see CheckboxActions component)
https://github.com/weaveworks/weave-gitops/assets/65822698/abdb7cc5-1670-45f7-a535-ae51955837a3

@yiannistri
Removed source sync for image update automations:

Default sync behavior will be addressed in a separate PR as discussed in standup 🌞