java.lang.RuntimeException: Duplicate class com.google.common.util.concurrent.ListenableFuture found in modules jetified-guava-20.0.jar (com.google.guava:guava:20.0) and jetified-listenablefuture-1.0.jar (com.google.guava:listenablefuture:1.0)[BUG]

Describe the bug
java.lang.RuntimeException: Duplicate class com.google.common.util.concurrent.ListenableFuture found in modules jetified-guava-20.0.jar (com.google.guava:guava:20.0) and jetified-listenablefuture-1.0.jar (com.google.guava:listenablefuture:1.0)
not able to run app , getting this error with version youtube_player_flutter: 7.0.0+7

android/app/build.gradle
dependencies {
implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
implementation 'com.google.guava:listenablefuture:9999.0-empty-to-avoid-conflict-with-guava'  //add this line

hi, i resolve the problem by add below line in app/build.gradle..
configurations.all{
exclude group: 'com.google.guava', module: 'listenablefuture'
}
my system function
System:
OS: Windows 10 10.0.22000
CPU: (4) x64 Intel(R) Core(TM) i3-1005G1 CPU @ 1.20GHz
Memory: 699.67 MB / 7.75 GB
Binaries:
Node: 14.18.0 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.15 - ~\AppData\Roaming\npm\yarn.CMD
npm: 6.14.15 - C:\Program Files\nodejs\npm.CMD
Watchman: 20210110.135312.0 - C:\ProgramData\chocolatey\bin\watchman.EXE
react: 18.0.0 => 18.0.0
react-native: 0.69.0 => 0.69.0
I hope so your problem is solve by this adding.. thank you.


Update img2img request

Widget counterpart for https://github.com/huggingface/api-inference-community/pull/244

the prompt parameter is already on prod, so this can be merge with either change for (inputs->image or image->inputs), ps still depends on https://github.com/huggingface/api-inference-community/pull/246


Adaptive Card mark down parser fails to indent and add bullets to markdown text

Platform
What platform is your issue or question related to? (Delete other platforms).

Android
iOS

Author or host
Host
If you're an author, who are you sending cards to?
1.1/1.0.3
What version are you using? Ex: NuGet 1.0.2, or latest master, etc...
Issue
Adaptive Card mark down parser fails to indent and add bullets to markdown text
{
"type": "AdaptiveCard",
"body": [
{
"type": "Container",
"items": [
{
"type": "Image",
"url": "https://cixdoq.bn.files.1drv.com/y4mswnHfii0b0TH6lDflshlunlPiT0Xu8_Ncrbjr3bSFfCF8gV8IyIDUl8zA8rZWFgryCXQ6gJ-JImjNfLkM8NgQIxqsbeIgI9vwcNRg4GoWSPchAjX9beEYdncTExDslPlkAULCcy67-D7qvAaIGVnY38JoNekYOiKlUqp68gxZ2EINJIu_LN-PdBzPtlkpnGC?width=1373&height=242&cropmode=none"
},
{
"type": "TextBlock",
"weight": "Bolder",
"text": "Reach people right where they collaborate. "
},
{
"type": "TextBlock",
"text": "Get the message out to employees using Microsoft Teams. Send announcements to a set of employees, stores, roles or locations in one or more channels or individually.\nUsing this app, you can:",
"isSubtle": true,
"wrap": true
},
{
"type": "TextBlock",
"size": "Small",
"text": "* Collaborate and communicate with large employee groups\n* Target announcements via 1:1 chats for select employees\n* Post in Channels to encourage discussion and feedback\n* Deliver announcements to desktop, web clients or mobile clients of Microsoft Teams  – wherever users are\n* Track and report employee engagement on what you post\n* Track and report employee’s “read receipt” if requested ",
"isSubtle": true,
"wrap": true
},
{
"type": "TextBlock",
"text": "Take your pick to get started:",
"isSubtle": true
}
]
}
],
"actions": [
{
"type": "Action.Submit",
"id": "createmessage",
"title": "📢 Create Message"
},
{
"type": "Action.Submit",
"id": "showscheduled",
"title": "⏱ View Scheduled"
},
{
"type": "Action.Submit",
"id": "viewall",
"title": "📄 View All"
},
{
"type": "Action.Submit",
"id": "viewstatistics",
"title": "👁🗨 View Statistics"
},
{
"type": "Action.Submit",
"id": "adminpanel",
"title": "⚙ Admin Panel"
}
],
"$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
"version": "1.0"
}

You have to use - for bullet list instead of *
(https://docs.microsoft.com/en-us/adaptive-cards/authoring-cards/text-features#markdown)

@BalajiR , aer you saying this is by-design? Also @nesalang can you take a look real quick and see if this is something related to and potentially addressed with changes for https://github.com/Microsoft/AdaptiveCards/issues/2651 at all?

@shalinijoshi19 @BalajiR is correct. it's by design.

Need #MsftTeams-Integration label @shalinijoshi19

Hi everyone, this was fixed in #4175 and will be available in the next 1.2.x patch at the end of the month


Getting error : unsupported operand type(s) for /: 'int' and 'RandomForestClassifier'

While running rf.fit(X, y, feature_names=features) in your github code I am getting below error,
: unsupported operand type(s) for /: 'int' and 'RandomForestClassifier'

+1

1

I can't replicate this, how is your RuleFit(...) instantiated?

I know it's probably way too late, but I found what is causing the problem.
The arguments in the initiator of the class RuleFit are in the wrong order:
def __init__(self,tree_size=4,sample_fract='default',max_rules=2000,
                 memory_par=0.01,
                 tree_generator=None,
                rfmode='regress',lin_trim_quantile=0.025,
                lin_standardise=True, exp_rand_tree_size=True,
                model_type='rl',Cs=None,cv=3,random_state=None):

If you create an instance of RuleFit, the first argument (your chosen classifier) is written into self.tree_size, which is supposed to be an integer, hence the the error message.
A fix to your problem could look like this:
clf = RandomForestClassifier(n_estimators=500, max_depth=10)
rf = RuleFit(tree_generator=clf, rfmode="helloworld")
rf.fit(X.to_numpy(), y, feature_names=features)

where X is an instance of pd.DataFrame and rfmode has to be set to something other than "regress", as otherwise a regressor is expected.
Took me some time to read through the source code and trace it down. Overall, the package does not seem to be polished, but I will keep experimenting.


1 eye for of 1 eye. it is a palindrome, test mark it as false.

Challenge Check for Palindromes has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_1) AppleWebKit/602.2.14 (KHTML, like Gecko) Version/10.0.1 Safari/602.2.14.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

function palindrome(str) {
  // Good luck!
  var pattern = /(\W)*(\d)*(_)*/g;
  var rep = str.replace(pattern, '').toLowerCase().split('');

  for(var i = 0, j = rep.length - 1; i < rep.length; i++, j--) {
    if(rep[i] !== rep[j]) {
      return false;
    }
  }

  return true;
}

palindrome("1 eye for of 1 eye.");

@Crizstian Thank you for reporting this issue. 1 eye for of 1 eye is not a palindrome.
var notPalindrome = "1 eye for of 1 eye";
notPalindrome.split("").reverse().join(""); // Returns "eye 1 fo rof eye 1"

This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "help" button on the challenge on Free Code Camp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding!

My mistake, i didn't read well the exercise, in the regex pattern i was taking out the numbers to.
with the numbers included in fact is not a palindrome.
thanks.


VS Code doesn't recognize project files

From @halcwb on July 7, 2016 23:6

VSCode Version: 1.3.0
OS Version: OS X 10.11.5

Steps to Reproduce:

clone http://github.com/halcwb/GenUnitApp.git
cd GenUnitApp
code .

Expect in the statusbar detection of 2 projects.
No projects are detected.
Copied from original issue: Microsoft/vscode#8861

Looking at GenUnitApp, these appear to be F# project files, not C#. Unfortunately, OmniSharp doesn't support F#. @halcwb: have you tried the Ionide extension for VS Code to add F# project support?


New package: MyProject v1.0.0-DEV

Registering package: MyProject
Repository: https://github.com/test-cc-maker/MyProject.jl
Created by: @test-cc-maker
Version: v1.0.0-DEV
Commit:<PHONE_NUMBER>04d247c14d58f981a848ee96734c2f
Git reference: HEAD
Description: cdsvdfvfd

Please don't open more test PRs (https://github.com/JuliaRegistries/General/pull/115114).


Remove button styling on filter

Fixes https://github.com/Microsoft/azuredatastudio/issues/5358

@anthonydresser Not sure what you mean there. I checked and the theming was working. Am I missing something?

Previously you added button stylers in the headerfilter code
https://github.com/Microsoft/azuredatastudio/commit/e870a309c0bdec9acfeb15cbbc6a3ffaa9478106#diff-f2838866a9f0bd6a2093b0469af7047fR212
presumably because there was something about them that wasn't natively theme aware. I'm not familiar with that ui so I can't tell you why, but those buttons generally need to have a styler attached.

@anthonydresser Is there a reason we removed that? What's the alternative to it then? Also, does https://github.com/Microsoft/azuredatastudio/blob/master/src/sql/base/browser/ui/table/plugins/headerFilter.plugin.ts#L240 take care of it?

/base/ shouldn't access anything other than /base/. In order to attach the button stylers directly in the header filter you would need to import /platform/ which would break that rule. styling the buttons as part of the header filter style function would fix it (assuming the attach button styler is called on the header filter) but the problem is that when you create header filter class the actual buttons haven't been created yet, causing an undefined access error.

@anthonydresser I don't see those issues on dev. Do those issues show up when packaged?

I'm assuming the bug references here is a null ref exception

Updated fix here - https://github.com/Microsoft/azuredatastudio/pull/5380


rc_moving_average_filter() dt argument should be double

line 681 in filter.c should be:
int rc_filter_moving_average(rc_filter_t* f, int samples, double dt)
otherwise the function fails to initialize the filter if dt < 1.0
the function prototype in the header file needs to be changed as well.

thanks! fixed in 64713b74d04fca1484f0ce290cbcebb036188890


add hyperbridge

So maybe add tarball support too? Why not both :v:

Ok I will work on this today

It would be nice to get the new revision ready tomorrow. Should we merge this without tarball support or is this on the way?

If this version works, using Docker, we can merge. No need to wait for tarball support if it's blocking a revision release.


Furnace Smelts only one item.

If there is an item in the output slot the furnace stops cooking food, ores, or anything like that.
Only works if FF is installed alone.
It's probably a conflict with a mod
The issue is in The SM Modpack + and other modpacks that have fast furnace installed
You can Download the SM Modpack to see it

it's a bug in custom npc's

But how can it be fixed?

I am interested in fixing this issue.
Just tell me how you found out it was CustomNPCs and i will try to fix it.

Ask Shadows-of-Fire he said it was bug with CustomNPCs


Project Sharing

Pull Request Description
Enso will now associate with two file extensions:

.enso — Enso source file.

If the source file belongs to a project under the Project Manager-managed directory, it will be opened.
If the source file belongs to a project located elsewhere, it will be imported into the PM-managed directory and opened;
otherwise, opening the .enseo file will fail. (e.g, loose source file without any project)

.enso-project — Enso project bundle, i.e. tar.gz archive containing a compressed Enso project directory.

it will be imported under PM-managed directory, if needed a unique directory name shall be generated.

Important Notes
On Windows the NSIS installer is expected to handle the file associations.
On macOS the file associations are expected to be set up after the first time Enso is started,
On Linux, the file associations are not supported yet.
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

Btw, I adore how well the code is commented. Amazing job with it ❤


[FormBuilderTextFieldl]: Separate labels and hintText from decoration

Is there an existing issue for this?

[X] I have searched the existing issues

Package/Plugin version
1
What you'd like to happen
It would be nice to have the TextField labels separate from decoration.
I'm trying to build a dynamic form and would like to build a template method/theme for layout purposes but currently cannot do that as every form field needs a different label and hint text.
 It either needs a way to override the parameter so you can add a label parameter or move the labels outside of InputDecoration.

InputDecoration textBoxLayout() {
  return InputDecoration(
    label: const Placeholder(),
    hintText: '$Placeholder()',
    constraints: const BoxConstraints(maxWidth: 1000, minHeight: 50, maxHeight: 300),
    border: const OutlineInputBorder(
        gapPadding: 1, borderRadius: BorderRadius.all(Radius.circular(10))),
    floatingLabelAlignment: FloatingLabelAlignment.start,
    floatingLabelBehavior: FloatingLabelBehavior.auto,
  );
}

Alternatives you've considered
No response
Aditional information
No response

Maybe you can use the method copyWith from property decoration on formField


git-trim is too verbose about remote references

Check your version before submitting the bug
$ git trim --version
git-trim 0.4.2

Describe the bug
When you work in a repository with many other collaborators, the git-trim output looks like this:
Branches that will remain:
  local branches:
    foo # here are my branches I'm interested in
    bar
  remote references:
    origin/blabla1 # here is a huge list of remote branches from all collaborators
    origin/blabla2

The list of remote references is so long that the "local branches" part is actually not visible unless I scroll up. However, in this kind of repositories, I'm only interested in removing my own branches, and not in all remote ones.
To Reproduce

Clone any repository with multiple branches
git trim --delete merged-local,local

Expected behavior
Remote references are not listed.
Actual behaviour
Remote references are listed, although I explicitly specified that I have no intention to clean them up.

I agree, it would be nice to have a config option to hide the remote references (or maybe only give the number of them?), and/or an option to limit the number of remote references shown?

Add my +1 here as well.


TASK [ceph-nfs : get keys from monitors]   The system got hang and become unresponsive with below error message

task path: /opt/ceph/ceph-ansible-master/roles/ceph-nfs/tasks/pre_requisite_non_container.yml:49
Using module file /usr/local/lib/python2.7/dist-packages/ansible/modules/commands/command.py
Pipelining is enabled.
<m-ttnkha.b.com> ESTABLISH SSH CONNECTION FOR USER: None
<m-ttnkha.b.com> SSH: EXEC ssh -o ControlMaster=auto -o ControlPersist=600s -o PreferredAuthentications=publickey -o StrictHostKeyChecking=no -o KbdInteractiveAuthentication=no -o PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey -o PasswordAuthentication=no -o ConnectTimeout=60 -o ControlPath=/root/.ansible/cp/%h-%r-%p clm-pun-ttnkha.bmc.com '/bin/sh -c '"'"'sudo -H -S -n  -u root /bin/sh -c '"'"'"'"'"'"'"'"'echo BECOME-SUCCESS-jdozhkkkedupmixcaapzlurwjlbmadbo ; /usr/bin/python'"'"'"'"'"'"'"'"' && sleep 0'"'"''
Escalation succeeded

This issue definitely lacks information.


CDN?

Hey have you thought about using a CDN for JsPsych versions? I'm working on experiment factory version II, and part of that is removing the dependency on our own expfactory-battery. It's really just acting as a CDN for experiment files! For now I'll just store jspsych with each experiment, but I thought it might make sense to chat about this and possibly get it up and running in a duplicated / "real" CDN.

I resisted initially, but I think that the time has come. I don't know much about CDN providers or deployment. Do you have any suggestions?

Unfortunately the most robust ones are going to charge you, for example maxcdn and Amazon Cloud Front (which has a free tier but I think only one year). From what I've seen of the JavaScript CDNs. MaxCDN used to offer a CDN just for open source software, and it looks like it's now called jsdelivr so that might be worth a try. The one that I use most often (which is javascript flavored) is cdnjs. For both, submission ("request a lib") is just via a PR to their repo. Those would be my suggestions as they are featured on maxcdn's open source page!

You can use https://www.jsdelivr.com/package/gh/jspsych/jsPsych
Everything is fully automated now, no manual submission needed.


Accessibility - Italics use an emphasis tag

Description
The only way I found to italicise some texts is the button “I” in the text editors.
But it just tags the text with an <em> without any semantic differentiation, and which means emphasis. I could not find a way to italicise a law or a title with <cite> for example.
Expected behaviour
Would it be possible to not use an emphasis semantic to italicise (as this is not the same than emphasis), and to provide a way to <cite> or <i> texts.
Otherwise this could result to a WCAG failure of SC 1.3.1.
Additional context
Reported in Freshdesk ticket 18070.

My understanding of this ticket is the user is asking us to add the ability cite and quote text.
To do this we would need to add new functionality for this. Probably as a button that semantically (and visually?) cites text and a button that quotes text. The behaviour of citing would be similar to em. The behaviour for quoting would similar to an anchor by taking a url to reference.
For design we'd probably want to think about this. Adding two new buttons may add a bit of confusion clutter to the toolbar. Maybe we could add an advanced mode or something to toggle on to show these and future "advanced" buttons? Also we'd need to style the buttons and also probably style the outputted html.
For example, here is an idea of what two more buttons would roughly look like in the editor:

@srtalbot to circle back with the client to confirm assumptions on requirements


Adding  a few more alt exhchange path/pairs to ccxt

Just some nits.
And could you address the pylint error?

I fixed the line too long error via black, but there was also this error at the bottom of the report:

src/dali/plugin/pair_converter/ccxt.py:90:0: C0301: Line too long (199/160) (line-too-long)

Not sure if this will show up again or not, but the closest thing I found was this closed issue in the pylint dev repo, which says this issue was resolved.
There was a mypy error that came after clearing the rest of the issues. I force pushed a rebase to take out the duplicate coinbase class def in the stub. That test just needs to be rerun (I ran it locally and it passed).
The alphabetical ordering (in addition to the one in the imports that you didn't mention in your review) was also pushed. Let me know if you need anything else on this.


[CLI]: Error when not using the defined layer in call method in Keras model with WandB integration

Describe the bug

When defining a new Keras model and integrating it with WandB (Weights and Biases), I encountered an error related to a specific layer. The error message I received is as follows:

File "/miniconda3/envs/tf/lib/python3.9/site-packages/wandb/data_types.py", line 1504, in from_keras
    node = Node.from_keras(layers[i])
  File "/miniconda3/envs/tf/lib/python3.9/site-packages/wandb/data_types.py", line 1731, in from_keras
    node.num_parameters = layer.count_params()
ValueError: You tried to call `count_params` on layer conv1d, but the layer isn't built. You can build it manually via: `conv1d.build(batch_input_shape)`.

Steps to Reproduce:

Define a new Keras model.
Include a specific layer within the __init__ method of the model.
Do not use the aforementioned layer in the call method of the model.
Integrate the Keras model with WandB for tracking.

Expected Behavior:
The model should integrate with WandB without any errors, even if there are layers defined in the __init__ method that is not used in the call method.
Actual Behavior:
The error mentioned above is raised when trying to integrate the Keras model with WandB.
Code Snippet:

import tensorflow as tf
import wandb

class MyModel(tf.keras.Model):
    def __init__(self):
        super(MyModel, self).__init__()
        self.conv1d = tf.keras.layers.Conv1D(64, kernel_size=3, activation='relu')
        # Other layers...

    def call(self, inputs):
        # Only using other layers, not `conv1d`
        return inputs

# WandB initialization
wandb.init(project="my-project", mode='disabled')
config = wandb.config

# Create and integrate the model
model = MyModel()

wandb_callback = wandb.keras.WandbCallback(
    monitor="val_loss",
    verbose=0,
    mode="min",
    save_model=False
)

x_train, y_train = tf.random.uniform([100, 10]), tf.random.uniform([100, 10])
wandb.config.update(config)  # Update config with any hyperparameters
model.compile(optimizer='adam', loss='mse')
model.fit(x_train, y_train, epochs=5, callbacks=[wandb_callback])
wandb.log({"example_metric": 0.85})
wandb.finish()

Please let me know if you need any further information or clarification.
Additional Files
No response
Environment
WandB version: 0.15.8
OS: Ubuntu 22.04
Python version: 3.9.16
Versions of relevant libraries:
TensorFlow Version: [2.12.0]
Additional Context
No response

Hello! Thank you for the reproducible code snippet! I was able to reproduce the error. I will reporting this bug to our SDK team for them to take a look at.

Hey @salmanmohebi this is not a W&B issue. I think your question was resolved in my comment to your SO question here. Someone also gave a detailed a answer stating the issue with the model definition.

Hi, @ayulockin I guess if we assume that W&B expects to receive properly built Keras models, you are right and this is not a W&B-specific issue.
Thanks for the comment, I will close the issue.

WandB Internal User commented:
salmanmohebi commented:
Hi, @ayulockin I guess if we assume that W&B expects to receive properly built Keras models, you are right and this is not a W&B-specific issue.
Thanks for the comment, I will close the issue.


[Bug]: shopify theme pull --only templates/*.json flag doesn't pull new files

The shopify theme pull --only templates/*.json command doesn't download new files.
Expected behavior
The shopify theme pull -t <theme_id> --only templates/*.json should download new files when they are created in the code editor.
Actual behavior
The shopify theme pull -t <theme_id> --only templates/*.json doesn't download the new file.
Reproduction steps

Create a new file at the admin code editor
shopify theme pull -t <theme_id> --only templates/*.json (notice nothing is downloaded)
shopify theme pull -t <theme_id> (notice the new file is downloaded)

Operating System
macOS
Shopify CLI version (check your project's package.json if you're not sure)
3.49.5
Shell
zsh
Node version (run node -v if you're not sure)
v20.2.0
What language and version are you using in your application?
3.1.2

Closing this issue because when users use --only templates/*, the CLI expands to local files because the shell performs the expansion before the command is run.
If users want to download files remotely, they may pass an expression as a regex to prevent the shell from expanding it. For example, to download only JSON files from the templates directory, use:
--only "templates/.*\.json"

This ensures the command applies the wildcard pattern to the remote files.

Closing this issue because when users use --only templates/*, the CLI expands to local files because the shell performs the expansion before the command is run.
If users want to download files remotely, they may pass an expression as a regex to prevent the shell from expanding it. For example, to download only JSON files from the templates directory, use:
--only "templates/.*.json"
This ensures the command applies the wildcard pattern to the remote files.

I'm not sure if this is working, I'm having an issue with templates created in the editor working, so basically all alternative templates, for example collection.champagne-magnum.json.
I've tried using the following command in my package.json but it isn't being pulled down.
"pull:content": "shopify theme pull --store=the-bottle-club --only config/settings_data.json --only \"templates/.*\\.json\" --only templates/customers/*.json --only templates/metaobject/*.json --only locales/*.json --only sections/*.json


Support for [CHIP-2021-03: Bigger Script Integers]

Link to the CHIP: https://gitlab.com/GeneralProtocols/research/chips/-/blob/master/CHIP-2021-02-Bigger-Script-Integers.md
No action seems to be required for bigger integer support on the compiler side. However on the SDK side JS numbers are limited to 53 bytes (versus 64 bytes of the new BCH big integers). So for that we need to allow the use of JS bigint at any place where you can currently provide number parameters.
The second part of the CHIP is the enabling of OP_MUL, so the * operator needs to be added to CashScript.

This has been implemented in v0.7.0-next.0. If some issues with it come up I'll have the time to fix that before May, if no issues come up, I will release v0.7.0 once the May 2022 upgrade hits mainnet.

v0.7.0 has been released for mainnet support of bigger integers and OP_MUL


Allow specifying release notes in PR description directly

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[x] Internal Infrastructure Improvements

Release Note Description
Allow specifying release notes in PR description directly

We should also communicate this to the vendors/update the docks they're using

I don't see much context. What's the motivation for this change? Is it just a preference in how ProdCon wants to handle changes?

Ah, nevermind, I found the Teams chat where this is being discussed.


Should KeyboardKit have a register of all locales?

If so, we can perhaps use this one here:
https://gist.github.com/jacobbubu/1836273

I close this, since I don't think so.

I close this, since I don't think so.


Origins Launchpad

https://xd.adobe.com/view/c7be57bf-6539-4c16-71e7-3a30731169c0-b18f/

Hi @t-z-u could you please attach the zip files so I can see what is being designed? Thanks.

Here is a branch from Origins SOV sale: https://github.com/DistributedCollective/Sovryn-frontend/tree/feat/sale/src/app/containers/SalesPage . This is mostly for inspiration since the design has changed a lot + we use Tailwind for styling now but there is an example of how to work with the "Enter access code" page.
Some info regarding the "Enter access code" page from Victor:

When users enter "access code", it sends blockchain tx with it and mints NFT. When the NFT is in users' wallets, it allows them to go to the screen where they can buy the token.
In the Genesis case, there were different NFTs that allowed different amounts to purchase.
If a user already has NFT, there is no access code screen.
But in the Genesis case, we allowed users to still open that screen from the buy screen (in case they had code to a higher tier NFT).

Info from Tzu regarding the screen flow:

If users have already engaged wallets, there will be no "Engage wallet screen".
We don't want to show the second screen ("Enter access code") if there is no access code required (it should be the case in the second day of the Fish sale).


Ability to increase the maximum total

At the moment the maximum is set to 40 mm, it would be nice to be able to increase this value for others who has a higher daily total.
This will be optional.

This is has been added in v1.3.0


Fix erroneous test for XCode 15.1

With Xcode 15.3 installed:
/usr/bin/c++ --version
Apple clang version 15.0.0 (clang-15<IP_ADDRESS>)

CMake reports:
-- The C compiler identification is AppleClang <IP_ADDRESS>00309
-- The CXX compiler identification is AppleClang <IP_ADDRESS>00309

Hence this test does not work as intended:
if( ${CMAKE_CXX_COMPILER_VERSION} VERSION_GREATER_EQUAL "15.0.0" AND ${CMAKE_CXX_COMPILER_VERSION} VERSION_LESS "15.1")
To confirm my suspicions about how the version number is reported by Apple Clang I booted into an old install running Xcode 14.2:
/usr/bin/c++ --version
Apple clang version 14.0.0 (clang-14<IP_ADDRESS>)

which CMake reports as:
-- The C compiler identification is AppleClang <IP_ADDRESS>00029
-- The CXX compiler identification is AppleClang <IP_ADDRESS>00029

so the real version number of Clang is in the 4th part, and takes the format for 2 digits for each part, dropping the final part from the longer number reported from --version.
I think this revised test is OK, although I was unable to test against a relevant version of Xcode that needs the linker flags and change of deployment target.

btw I found this list of Xcode and corresponding Clang versions, which shows:
# Xcode 15.1 (15C65)
Apple clang version 15.0.0 (clang-15<IP_ADDRESS>.5)

so I think that the corresponding CMAKE_CXX_COMPILER_VERSION for this version would be <IP_ADDRESS>00100 and my updated version test should hold, as both the prior Xcode versions are:
# Xcode 15.0 (15A240d)
Apple clang version 15.0.0 (clang-15<IP_ADDRESS>)

which should show as <IP_ADDRESS>00040 (assuming there is logic to the Apple Clang version number!)

Thanks for looking into this! The change appears correct to me, but I'd prefer to wait for Paul to have a look before merging.

I’m on vaca right now but it also looks correct esp with the crazy cmake versions whic we guessed before Xcode 15.1 even shipped! Let’s merge it.


Indicator with letter shows up and never hides (TouchScrollBar)

After upgrade to latest version 13.2.1 TouchScrollBar shows letter and never hides. Can you look into it?

 I'll get it fixed shortly!

Thanks!


Update panoply to 4.6.0

Checklist

[x] The commit message includes the cask’s name and version.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

This is perfectly fine


Allow configurable minimum TLS version

As part of resolving #166, I'm modifying the connection logic used to establish client connections. This includes explicitly configuring the TLS settings for use with client.DialWithDialerTLS(), vs using the client.DialTLS() wrapper method (both from the emersion/go-imap package).
As part of this work, I'm setting tls.Config.MinVersion to TLS v1.2, vs the default TLS v1.0 value. This resolves a gosec warning.
Ideally this would be user configurable, so I'm filing this issue as a reminder to add this support in the future.

See also:

https://github.com/go-redis/redis/issues/1553
https://github.com/RedHatInsights/insights-operator-controller/pull/381/files


Reimport tracking result with different post processing parameter

Requested feature from WIZARD at eroscripts:

Would it be possible to integrate some sort of “undo” function, where after adjusting the processing menu of the tracking generator and applying the results, to go back to that processing menu if you are not satisfied with the inputted data into the OFS ‘positions’ bar?

Required Implementation:
[ ] Button in Lua plugin to trigger post processing from last raw data.At import first delete the current existing data in the range of the new data.
[ ] CLI Args in MTFG for post processing from last raw data
[ ] Save raw data when tracking is completed

available with [v0.5.1(v0.5.1)


[docs] Investigate IE 11 crash

Investigating #14774
git bisect good ba4332d00a5f951cfc33dcffa5924cc7999b8f05
# first bad commit: [440c9f6320d669319b5a3524b2705d73e795e698] [core] Upgrade dependencies and devDependencies (#14493)

440c9f6320d669319b5a3524b2705d73e795e698 regenerated the whole lockfile. Might be just next might be some other dependency. Might be a good idea to branch of ba4332d00a5f951cfc33dcffa5924cc7999b8f05 and increment by canaries instead of majors.
Already spent a couple of hours on this issue. Working with next.js is pretty frustrating so I'll doubt I spend much more time on the issue. Maybe it will fix itself someday.
No idea why netlify is crashing. It's working locally.

I have looked deeper into the problem. I believe that we just have to wait for the release of
I have found the following reproduction:
🔴 KO
import 'docs/src/modules/components/bootstrap';
// --- Post bootstrap -----
import React from 'react';
import { makeStyles } from '@material-ui/styles';

const useStyles = makeStyles({
  root: {
//    flex: '1 0 100%',
  },
});

const B = () => {
  useStyles();
  return
wtf
;
};

export default () => ;

✳ OK
```jsx
import 'docs/src/modules/components/bootstrap';
// --- Post bootstrap -----
import React from 'react';
import { makeStyles } from '@material-ui/styles';

const useStyles = makeStyles({
  root: {
    flex: '1 0 100%',
  },
});

const B = () => {
  useStyles();
  return
wtf
;
};

export default () => ;

Have you tried building this branch locally or any other idea why netlify is failing?
I guess we should at least add the Map/Set polyfills. I just recently encountered an issue with the codesandbox-client. IE11 essentially has a non-spec compliant implementation where someMap.set().set() is failing because they return undefined from Map.prototype.set.
I remember that we also had some issues where we had to remove new Set([iterable]).

@eps1lon I have tried this pull request. My workflow was: remove all the pages from /pages at the exception of index.js. Do a dichotomy to find the source of the problem. The minimal changes required was: do no mount any children, remove the style rules body.

Using the IE 11 debugger is tricky. I had to pause on all the exception. There are quite exceptions thrown by feature detection logics.

Using the IE 11 debugger is tricky. I had to pause on all the exception. There are quite a few exceptions thrown by feature detection logic that should be ignored. The first meaninfuly error reported is the JSS startWith issue.

I feel you. I wish we could disable next.js error pages so that we could actually catch the exception and not just a logged error from componentDidCatch.
But your debug approach was a bit different. While jss might be a issue it was not the original issue. The major version upgrade of next.js was the first one to introduce a change. I never saw an error in IE11 related to startsWith on that commit.
Does the full site work with all flex usages removed?

Upgrading JSS fixes the problem: #15034. Do you want to keep pushing in this pull request direction?

We can close. I'll keep an eye on the polyfill issue that's not working in netlify. The broken Map/Set polyfill might cause trouble in the future.

Yes, it has already caused trouble in the past, at least for me in Apollo core with IE 11.


feat(Data/List/TFAE): version of forall_congr' for TFAE

Zulip

Should it be List.TFAE.forall so that we can write := .forall fun i => _? Also, could you please add a version for Exists?

Should it be List.TFAE.forall so that we can write := .forall fun i => _? Also, could you please add a version for Exists?

It doesn't seem to work, but I'm not familiar enough with this new syntax to understand why it would/wouldn't work.

bors r+
Thanks!


Upgrade to Grafana 9.0.6

Codecov Report

Merging #34 (f71475a) into main (d79f21b) will not change coverage.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##             main      #34   +/-   ##
=======================================
  Coverage   55.26%   55.26%
=======================================
  Files          39       39
  Lines         418      418
  Branches       57       57
=======================================
  Hits          231      231
  Misses        172      172
  Partials       15       15

Impacted Files
Coverage Δ

src/components/Config/Config.tsx
100.00% <ø> (ø)

src/constants.ts
100.00% <ø> (ø)

.../volkovlabs-balenasupervisor-datasource/api/api.ts
22.80% <25.00%> (ø)

...-datasource/components/QueryEditor/QueryEditor.tsx
51.61% <100.00%> (ø)

...abs-balenasupervisor-datasource/constants/query.ts
100.00% <100.00%> (ø)

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


Add SKIP_ES_TESTS option to skip ES tests until external ES image is …

…available for P&Z

@kevinearls @objectiser @jpkrohling can you please review this?
@madeelrh wants to skip external elastisecarch tests on the p/z environment. As they do not have external es image for p/z environment.

@jkandasa @kevinearls there are a couple of things that concern me about this change - the test suite being disabled also include tests for spark dependencies, index clearer, etc - and if using external ES is an issue at the moment, wouldn't the other test suites that rely on the external ES also need to be disabled (i.e. streaming, auto-scale, examples2, multi-instance)? which would effectively disable most of the e2e tests.
This may require a more extreme refactor, to extract the way in which the Jaeger instance is created (whether auto or self provisioned ES and possibly Kafka???), from the tests being performed on them?

@jkandasa Do you know if self-prov ES instances work for P&Z?  If so I think it would be fairly simple to update most of the tests that currently use the static instance to use self prov instances.  I can write up an example if you think this would work.

@madeelibm can you please confirm?
Do you have RH ES operator on both P/Z environment?

@jkandasa as discussed with you before, we do have RH ES operator available for both P&Z. The ES instance created by the Operator can be accessed by an example CR, which uses certificates to access the ES instance. However, with the Jaeger self provisioned test the ES instance doesn't become Green. Some problem with node discovery, which doesn't happen, if I create ES instance manually from UI i.e. Installed Operators -> Red Hat OpenShift Jaeger -> Create Jaeger.

@madeelrh  I have a few questions:

Are you saying that if you create a self-provisioned Jaeger instance from the operator UI it works correctly?
Can you run the e2e-self-provisioned test with SAVE_LOGS=true and then attach (or email to me) the logs it creates as well as the logs for the Jaeger and ElasticSearch operators?

@kevinearls I am little bit ahead with debugging this issue. The TestSelfProvisionedESSmokeTest passed now with following changes:
--- a/test/e2e/self_provisioned_elasticsearch_test.go
+++ b/test/e2e/self_provisioned_elasticsearch_test.go
@@ -237,8 +237,10 @@ func getJaegerSimpleProd(instanceName string) *v1.Jaeger {
Elasticsearch: v1.ElasticsearchSpec{
NodeCount: 1,
Resources: &corev1.ResourceRequirements{

                                          Limits:   corev1.ResourceList{corev1.ResourceMemory: resource.MustParse("1Gi")},

                                          Requests: corev1.ResourceList{corev1.ResourceMemory: resource.MustParse("1Gi")},

                                          Limits: corev1.ResourceList{corev1.ResourceMemory: resource.MustParse("4Gi")},

                                          Requests: corev1.ResourceList{

                                                  corev1.ResourceCPU:    resource.MustParse("4"),

                                                  corev1.ResourceMemory: resource.MustParse("512Mi")},

However, TestIncreasingReplicas still fails and we need 4.6 RH ES operator to pass Replicas test.
With one ES node, we are good with self provisioned tests. Could you change the rest of Jaeger test to use the self provisioned instance?
Answer to your question:

Yes
I will email you logs.

I don't think so since the automation totally changed @frzifus

ok thanks for clarification @iblancasa


timeseries/spectre (Fix issue 269)

migrates the multivariate tools that had been stashed in mth5/utils/ during initial development to timeseries/spectre.
Renamed this module from fc_tools.py to multiple_station.py
brings the Spectrogam class from aurora into mth5 (this is a fairly general class that can be used to wrangle FCs, and should not be aurora-specific)
update imports accordingly

@kujaku11 the PR to bring this into patches is an FYI to clarify these changes from what is already in patches.  When you approve please consider merging patches into main as well to allow aurora to import these methods from mth5 moving forward.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 54.73684% with 43 lines in your changes missing coverage. Please review.

Project coverage is 62.47%. Comparing base (113f1b0) to head (2c131cc).

Files with missing lines
Patch %
Lines

mth5/timeseries/spectre/spectrogram.py
33.87%
41 Missing :warning:

mth5/timeseries/spectre/multiple_station.py
80.00%
1 Missing :warning:

tests/timeseries/spectre/test_spectrogram.py
93.33%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           patches     #270      +/-   ##
===========================================
- Coverage    62.54%   62.47%   -0.07%
===========================================
  Files          164      167       +3
  Lines        17523    17606      +83
===========================================
+ Hits         10959    11000      +41
- Misses        6564     6606      +42

Flag
Coverage Δ

tests
62.47% <54.73%> (-0.07%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add forward rule to account for default DENY in FORWARD

https://github.com/rancher/rancher/issues/4722


Remove Battery Saver effect on Background Access Status

Battery Saver being on does not cause Background Access Status to be "DeniedDueToSystemPolicy", changing to update that. Registration of a background task can still occur while Battery Saver is engaged, changing to update that as well.

@cc-cortes,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot