Ecommerce trends Analysis

Pull Request for ML-Crate 💡
Issue Title:  #652

Info about the related issue (Aim of the project) :  to analyze ecommer trends data using ml algorithms
Name:  Aditya D
Email ID for further communication<EMAIL_ADDRESS>GitHub ID:  https://www.github.com/adi271001
Idenitfy yourself:  VSOC Contributor

Closes: #issue number that will be closed through this PR #652
Describe the add-ons or changes you've made 📃
Give a clear description of what have you added or modifications made
Added folders containing models , images , results , requirements.txt
Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested? ⚙
Describe how it has been tested
divided dataset into test and train , evaluated for test set
Describe how have you verified the changes made
i have verified the model for overfitting and underfitting via graphs
Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

While writing the Conclusion use bullet points, otherwise it's not looking that much eye-pleasing.

@abhisheks008 done please do check

While assigning your points in the VSOC, you email is not found there. @adi271001

While assigning your points in the VSOC, you email is not found there. @adi271001

Add your details in the Contributor-list channel in the VSOC server, otherwise your points will not going to be calculated from their end.

@abhisheks008 i have added the details in contributor list please inform me once points are added

@abhisheks008 i have added the details in contributor list please inform me once points are added

Added your points with respect to your email ID.


Show loading indicator in staking overview

UI Version: 0.10.2
Description:
Loading staking info (staked atoms) takes a long time (45s). We cache the results to improve UX. Still we need to show an indicator, that Voyager is still loading the data from chain.

i actually think for loading the staking overview (at 45s) we need to have an obtrusive loader. 🐌

while loading the first time. after that, it could be done unobtrusive don't you think?

waiting for cosmos/cosmos-sdk#2249 to be merged so that we won't require the loader


Can't fill a single Auth schema if there are multiple with same type

What happens?
I'm using the scalar ui from my angular application and i want to show and autofill api tokens for an openapi spec that has two mandatory authorization schemas. Both of them are of type 'apiKey' but with different names. Id like to autofill just one of them and keep the other one empty, but im only able to fill both of them or none. How do I configure it?
example schema:
securitySchema:
 X-Domain-ApiKey:
  in: header
  name: apikey1
  type: apiKey
 X-Domain-Host:
  in: header
  name: apikey2
  type: apiKey

example configuration:
authentication: {
 apiKey: {
 token: "xyz"
 }
}

What did you expect to happen?
I expected to fill just one of them, and not both of them
OpenAPI Document
No response

Once we update to the new auth this should be possible

Hello,
any update?

hello, sorry I was on vacation 😓 if I find some time I will try to get it in before the break!


Draft schema for short_citation of census["census_info"]["dataset"]

see https://czi-sci.slack.com/archives/C023Q1APASK/p1716245770988479 for reference.
Schema changes
[...]
Census table of CELLxGENE Discover datasets – census_obj["census_info"]["datasets"] – SOMADataFrame
All datasets used to build the Census MUST be included in a table modeled as a SOMADataFrame. Each row MUST correspond to an individual dataset with the following columns:

Column
Encoding
Description

citation
string
As defined in the CELLxGENE Discover API data schema.

collection_id
string

collection_name
string

collection_doi
string

collection_doi_label
string
Corssref response from DOI with format "last name of first author et al. (year) journal". For example "Lee et al. 2024 Cell"..

dataset_id
string
As defined in the CELLxGENE Discover API data schema.

dataset_title
string

dataset_h5ad_path
string
Relative path to the source h5ad file in the Census storage bucket.

dataset_total_cell_count
int
Total number of cells from the dataset included in the Census.

dataset_version_id
string
As defined in the CELLxGENE Discover API data schema.

[...]
Changelog
Version 2.1.0

Adds collection_doi_label to "Census table of CELLxGENE Discover datasets – census_obj["census_info"]["datasets"]"

[...]

Nit. I would not use "crossref" in the name. It's just the mechanism used to acquire publication metadata.
The citation is an internal design based on that metadata.
How about collection_doi_labelto differentiate from the dataset citation and to self-document that it's related to collection_doi.

That's a good name. Updated.

Updated since the string will be directly fetched from Discover API

Completed by #1200


could not find version that satisfies...

this is the error i get when trying to download:
C:\WINDOWS\system32>pip install torch==1.5.0+cpu torchvision==0.6.0+cpu -f https://download.pytorch.org/whl/torch_stable.html
Looking in links: https://download.pytorch.org/whl/torch_stable.html
ERROR: Could not find a version that satisfies the requirement torch==1.5.0+cpu (from versions: 0.1.2, 0.1.2.post1, 0.1.2.post2)
ERROR: No matching distribution found for torch==1.5.0+cpu

it looks like the installation documentation is out of date for installing torch, as the version specified isn't available at that git anymore.


other works in show do not show all the other works in a show

there seems to be a cap of how many works are show on an artwork page underneath other works in the show header
see this show from Trevor Bell http://cl.ly/image/30073u0u3p0Y
Compared to how many works are in the show here https://www.artsy.net/artwork/trevor-bell-bernina

This could be that we have an artworks limit and have to paginate.
Estimate: 1 day.

How would pagination work when there’s also a section underneath it? Like in @katarinabatina’s example there is the “Other Works by Author” section.


TRTEngineOP name conflict fix.

Fix the TRTEngineOp naming conflicts that could occur in some situations.
To fix such conflicts we are using a randomly generated 5-character's converter_ID which is stored as an attribute of the node. Now the resource manager is looking for such nodes using converter_ID and TRTEngineOp_X_Y instead of the previously used TF-TRT and TRTEngineOp_X_Y.

TRTEngineop name:
TF-TRT creates placeholder nodes in the graph with the type TRTEngineOp, and name TRTEngineOp_X_Y, where:

X is the number of models converted in the current process
Y is the number of TRT-compatible segments in the current graph.

The engines that correspond to the TRTEngineop nodes are stored in the resource manager, and they are lookup up based on the op name (TRTEngineOp_X_Y).
Problem:
Assume that the user has a workflow with two models:
input -> model_A -> model_B -> output

If the user converts model_A and model_B in a separate process, then in both cases the TRTEngineOp nodes would be called TRTEngineOp_0_Y. The names would clash while looking up the engines, therefore the user cannot load the two models in a single process.

This is still in draft status, please change the status when it is ready for review.

@drivanov Can you describe the solution in the PR description?

@bixia1 : I added a short description of the solution to the PR description.

We could generate a deterministic name the following way:
def generate_container_name(graph_def, rewriter_config):
    tf_string = 'TF' + tf.__version__
    trt_string = 'TRT' + (".".join([str(x) for x in _pywrap_py_utils.get_loaded_tensorrt_version()]))
    model_string = 'model' + str(hash(graph_def.SerializeToString()))
    conv_param_string = 'param' + str(hash(rewriter_config.SerializeToString()))
    return '_'.join([tf_string, trt_string, model_string, conv_param_string])

This could generate a name like this:
'TF2.9.1_TRT8.4.2_model8555381799981073735_param7256064327999100924'

We could use this as container name during engine lookup. The name would be stored as an attribute of the TrtEngineOp as it is currently proposed by the PR. This is now specific to model, conversion params, and TF-TRT version. What do you think @bixia1 and @DEKHTIARJonathan?

We could generate a deterministic name the following way:
def generate_container_name(graph_def, rewriter_config):
    tf_string = 'TF' + tf.__version__
    trt_string = 'TRT' + (".".join([str(x) for x in _pywrap_py_utils.get_loaded_tensorrt_version()]))
    model_string = 'model' + str(hash(graph_def.SerializeToString()))
    conv_param_string = 'param' + str(hash(rewriter_config.SerializeToString()))
    return '_'.join([tf_string, trt_string, model_string, conv_param_string])

This could generate a name like this:
'TF2.9.1_TRT8.4.2_model8555381799981073735_param7256064327999100924'

We could use this as container name during engine lookup. The name would be stored as an attribute of the TrtEngineOp as it is currently proposed by the PR. This is now specific to model, conversion params, and TF-TRT version. What do you think @bixia1 and @DEKHTIARJonathan?

I am fine with this or similar ideas, but that needs to be implemented in the C++ code as we have C++ API for converting graphs.

I'm seeing a failure here:
[.../tf2tensorrt/trt_convert_api_test.cc:255]
Expected equality of these values:
  ::tsl::OkStatus()
    Which is: OK
  (result.status())
    Which is: NOT_FOUND: Container TF-TRT does not exist. (Could not find resource: TF-TRT/TRTEngineOp_000_000)

Followed by a string of similar failures.
@drivanov, can you repro these problems on your side and address them?

@gbaned : I just resolved some merge conflicts on trt_convert.py. Hope this helps

@gbaned I should not review in Github right? NVIDIA should review here.

@gbaned I should not review in Github right? NVIDIA should review here.

Hi @poulsbo Sorry, I have assigned @pjannaty to review. Thank you!

Hi @pjannaty Can you please review this PR ? Thank you!

Hi @pjannaty Can you please review this PR ? Thank you!

Hi @pjannaty / @DEKHTIARJonathan Can you please review this PR ? Thank you!

Hi @pjannaty / @DEKHTIARJonathan Can you please review this PR ? Thank you!

Hi @pjannaty / @DEKHTIARJonathan Can you please review this PR ? Thank you!

Hi @pjannaty / @DEKHTIARJonathan Can you please review this PR ? Thank you!

Hi @pjannaty / @DEKHTIARJonathan Can you please review this PR ? Thank you!

Hi @pjannaty / @DEKHTIARJonathan Can you please review this PR ? Thank you!

Hi @pjannaty / @DEKHTIARJonathan Can you please review this PR ? Thank you!

Hi @pjannaty / @DEKHTIARJonathan Can you please review this PR ? Thank you!

Hi @drivanov, @pjannaty, @DEKHTIARJonathan  I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen! Thank you!


[Bug]: Text field placeholder is invisible when not focused

🕵 Is there an existing issue for this?

[X] I have searched the existing issues

What version of spark are you using?
Other (please specify it)
🐛 What happened?
Hi,
I'm using the TextField with a placeholder attribute and I've realized that the placeholder text is visible only when the field is focused, remaining invisible otherwise.

(To be more accurate, I'm using MultilineTextField, but maybe the bug applies to all versions of TextField)
Thanks
Relevant logcat output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

### Tasks

Hi @Sildian After investigating the issue and talking with Spark designers.
You must provide a label for any TextField component,
also this is for Accessibility reasons (It's better on TalkBack)
By doing so, this shall resolve your issue.
Thanks for reporting


Feat: add <Table.Cell> component

Add <Table.Cell> component
Enable <Table> inner border preview
Add <Table> margin support

@wyozi Let me know what I'm missing to make the <Table> node open to support Cell, but not as child and keep the rows as prop

Still missing styles details on the render for download

@wyozi  Let me know when you reviewed please 😁

Looks very good 👌 Just left a few notes

Thanks! Will make those for tomorrow 😁

LGTM! Thanks


Renaming The Infections Seeding Initialization

This PR clarifies naming for $I_{v:0}$, i.e. the seeded infections, by framing the variable for infections seeding in terms of "initialization" versus "seeding", which, in most instances, is perceived by users as the value for pseudo-random number generation reproducibility.

Thanks @AFg6K7h4fhy2! I think we should perform more renaming throughout. For instance, InfectionSeedingProcess should be InfectionInitializationProcess. In general, you should search the whole repo for seed and make sure it is only used in the context of RNG seeds. @dylanhmorris do you agree?

I agree. I held because I didn't want to decide unilaterally. Beyond InfectionSeedingProcess, I am thinking any other place where we have Infection and Seeding not in reference to pseudo-random number usage.

Thanks @AFg6K7h4fhy2! I think we should perform more renaming throughout. For instance, InfectionSeedingProcess should be InfectionInitializationProcess. In general, you should search the whole repo for seed and make sure it is only used in the context of RNG seeds. @dylanhmorris do you agree?

I do. @AFg6K7h4fhy2, I think you can assume initialization is the agreed-upon substitute for seeding throughout.

I do. @AFg6K7h4fhy2, I think you can assume initialization is the agreed-upon substitute for seeding throughout.

Sounds good, I will edit this after I get through #241

Thanks @AFg6K7h4fhy2! I prefer post_initialization_infections over post_initialized_infections.

Thanks @AFg6K7h4fhy2! I prefer post_initialization_infections over post_initialized_infections.
The documentation, for example docs/source/msei_reference/latent.rst, also needs to be changed.

Will do on post_initialized_infections → post_initialization_infections. Good catch on the ./docs/ changes. I think I was one level off when doing Command-Shift-F.

I think we also want to rename the InfectionInitializationMethods (e.g., SeedInfectionsExponentialGrowth -> InitializeInfectionsExponentialGrowth).

I think we also want to rename the InfectionInitializationMethods (e.g., SeedInfectionsExponentialGrowth -> InitializeInfectionsExponentialGrowth).

I was thinking that since they were subclasses the connection with seeding infections shouldn't be completely removed, but I see the point. Will change.


RuntimeError: The size of tensor a (2) must match the size of tensor b (50) at non-singleton dimension 1

When I load the ViT-B/32 model, like so,
clip_image_and_text_model, preprocess = clip.load("ViT-B/32", device=device)

And then construct a CIFAR100 dataloader using preprocess as the dataloader, I have input tensors of shape (batch size, 3, 32, 32). But when I try calling
clip_image_and_text_model.encode_image(input_tensor)

I get RuntimeError: The size of tensor a (2) must match the size of tensor b (50) at non-singleton dimension 1. The error is caused by line 224 in model.py:
x = x + self.positional_embedding.to(x.dtype)

At this line, x has shape (batch size, 2, 768) whereas self.positional_embedding has shape (batch size, 768).
I don't know what the correct behavior is to know whether x should not have dimension 1 or self.positional_embedding should have dimension 1 or something else entirely.
Could someone please clarify?

The comment on line 223 suggests that x should indeed be 3 dimensional, which suggests to me that the positional embedding is wrong. Is this correct?

Hey @RylanSchaeffer , I am finding a simillar problem when changing the input size of the model. Did you finally manage to find the proper solution?

@Evm7 unfortunately I can't remember :(

Facing exactly the same issue here

Found the solution. My problem was the size of the images: I had batches of dimension (16, 3, 32, 32) (16 images per batch, 3 channels, 32 height/width). Got it working when changed the Transforms to resize to dimension 224. So, final shape became (16, 3, 224, 224). Hope it helps!

Found the solution. My problem was the size of the images: I had batches of dimension (16, 3, 32, 32) (16 images per batch, 3 channels, 32 height/width). Got it working when changed the Transforms to resize to dimension 224. So, final shape became (16, 3, 224, 224). Hope it helps!

in other words, the input shape must be (B, 3 , 224, 224) ?


MB-6464 generalize filters

Description
Reduce code duplication by making a utility for converting URL query parameters into concrete QueryFilter types.
Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Messages

:book:
:link: MB-6464

  Generated by :no_entry_sign: dangerJS against 3f81e838fdf9198e1da6097e016f9537196802cd


FPS Drop when press aimbot trigger key

I have an RTX 3080, with CUDA Enabled
it fine when i dont press aimbot trigger key, get about 60fps on overlay
but when i press aimbot trigger key, it drop to 3fps, and keeping moving mouse.

kakahost @.***>於 2022年6月15日 週三，下午9:47寫道：

Don't think about it. You can't solve it, how author says. Just keep play.
—
Reply to this email directly, view it on GitHub
https://github.com/Zurek0x/NuremX/issues/10#issuecomment-1156497371, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AM7QYMMWTFTEMYT4LCIQWHLVPHNHZANCNFSM5YUVOMVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I can’t even play, drop to 2fps on overlay, and I can’t move mouse

That means that your CPU is bad

kakahost @.***>於 2022年6月16日 週四，上午12:11寫道：

That means that your CPU is bad
—
Reply to this email directly, view it on GitHub
https://github.com/Zurek0x/NuremX/issues/10#issuecomment-1156666301, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AM7QYML5BXUVLDIYXLRNZ43VPH6DDANCNFSM5YUVOMVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yooooooo, I just fix it, thx for ur reply tho

Notice About Performance
This script uses A LOT of RAM/CPU/GPU Power, You must have at least 3 GB of DDR3 Ram dedicated to the script, Note that also anything lower peformance then a GTX 1080 TI will have delayed peformance and have a lower Frame Count You also want at least a 3Ghz (or more) CPU with more then 3 Cores Minimum.


Only background in output, or asset if I delete background

I've been trying to generate background with 6 traits. But I've stumbled into problem that output only shows background or one asset. Files are 24x24 pixel and in a .png form transparent. I get no errors or anything like that. The weirdest part is that I can get on my laptop background + next trait but on my main only background or a trait if I deselect background.
How I should proceed?

I've been trying to generate background with 6 traits. But I've stumbled into problem that output only shows background or one asset. Files are 24x24 pixel and in a .png form transparent. I get no errors or anything like that. The weirdest part is that I can get on my laptop background + next trait but on my main only background or a trait if I deselect background.
How I should proceed?

Having the same issue. Any updates?


Finding WEBSOCKET_EVENT_DISCONNECTED Reason  (IDFGH-12034)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
All,
im using esp32 wroom and espidf v4
trying to make connection with websocket - it is working properly.
In case of, No internet, server not availabe, or certificate expire or not trusted CA.
i want to capture the exact reason. if there is any possibilities to capture the reasons.
static void websocket_event_handler(void *handler_args, esp_event_base_t base, int32_t event_id, void *event_data)
{
    esp_websocket_event_data_t *data = (esp_websocket_event_data_t *)event_data;
    char rx_buffer[128];

  switch (event_id)
    {
                case WEBSOCKET_EVENT_CONNECTED:
                ESP_LOGW(TAG, "WEBSKT_EVENT_CNCTD");
                websocket_connection_status=1;
                websocket_connection_cnt++;
                if(0 == get_con_sts(WSS_CLIENT)){
                    conn_sts_upd(WSS_CLIENT,1, (int32_t)client);
                    auth_sts_upd(WSS_CLIENT,1);
                }
                    break;

            case WEBSOCKET_EVENT_DISCONNECTED:

                ESP_LOGW(TAG, "WEBSKT_EVENT_DISCNCTD the server is unavailable");
                websocket_connection_status=0;
                uint32_t intf_pos=WSS_CLIENT;
                conn_sts_upd(WSS_CLIENT,0, (int32_t)client);
                queue_event_send(CTRLR_THREAD,CONNECTIVITY_NO,(void*)intf_pos,0,IPC_THREAD_CONTEXT,PLD_NOT_PRESENT,IPC_PLD_FREE_NOT_REQ,controller_queue);
                break;

        case WEBSOCKET_EVENT_DATA:
            if (data->op_code == 0x08 && data->data_len == 2) {
                ESP_LOGW(TAG, "Received closed message with code=%d", 256*data->data_ptr[0] + data->data_ptr[1]);
            } else {
                ESP_LOGW(TAG, "Total payload length=%d, data_len=%d, current payload offset=%d\r\n",\
                        data->payload_len, data->data_len, data->payload_offset);
            }
            memcpy((char *)rx_buffer,(char *)data->data_ptr, data->data_len+1);
            /* for(int i=0;i<data->data_len;i++)
               {
               printf("buffer[%d]=%d %X \n",i,rx_buffer[i],rx_buffer[i]);
               }*/
            rx_buffer[data->data_len] = 0;

            if((data->op_code!=0x0A) && (data->op_code!=0x09) && (data->op_code!=0x08))
            {
                rx_frame_process_hdlr((uint8_t*)rx_buffer,data->data_len,WSS_CLIENT,&frame_payload);
            }
            break;

        case WEBSOCKET_EVENT_ERROR:
            ESP_LOGE(TAG, "WEBSOCKET_EVENT_ERROR no action in this event");
            break;

        case WEBSOCKET_EVENT_MAX:
            ESP_LOGE(TAG, "WEBSOCKET_EVENT_MAX no action in this event");
            break;

        case WEBSOCKET_EVENT_CLOSED:
            ESP_LOGE(TAG, "The connection has been closed cleanly");
            break;
    }
}

@karthi151088 Updates: I've just noticed that you are using esp-idf v4. Unfortunately, the tls stack error handling feature is available in esp-idf version 5.0 and above. Following the release of version 5.0, the esp-websocket-client has transitioned from esp-idf to esp-protocols. And you can check the documentation there


Wrong BASE_URL in signup confirmation email

I have my production server setup for email as follows:
# ./settings-production.py
ACCOUNT_EMAIL_REQUIRED = True
ACCOUNT_USERNAME_REQUIRED = False
ACCOUNT_EMAIL_VERIFICATION = "optional"

EMAIL_BACKEND = "django.core.mail.backends.smtp.EmailBackend"
EMAIL_HOST = "smtp.gmail.com"
EMAIL_PORT = 587
EMAIL_USE_TLS = True
DEFAULT_FROM_EMAIL = "..."
EMAIL_HOST_PASSWORD = config.get("email", "EMAIL_HOST_PASSWORD")
EMAIL_HOST_USER = config.get("email", "EMAIL_HOST_USER")

And a url.py file that contains:
urlpatterns = [
    ...
    # auth
    path("auth/", include("dj_rest_auth.urls")),
    path("auth/register/", include("dj_rest_auth.registration.urls")),
    path(
        "auth/register/account-confirm-email/",
        VerifyEmailView.as_view(),
        name="account_email_verification_sent",
    ),
    path(
        "auth/password/reset/confirm/<slug:uidb64>/<slug:token>/",
        PasswordResetConfirmView.as_view(),
        name="password_reset_confirm",
    ),
    ...
]

And when I enter a change password request, I get the following email.

You're receiving this email because you requested a password reset for your user account at mysite.com.
Please go to the following page and choose a new password:
http://mysite.com/rest-auth/password/reset/confirm/MTI/ahaq56-227b8a354226bcf7e7da7a99fa00edb0/

However, when I enter a new user signup, I get a URL with a localhost base:

Hello from mysite.com!
You're receiving this e-mail because user me has given your e-mail address to register an account on mysite.com.
To confirm this is correct, go to http://localhost:8080/registration/account-confirm-email/OQ:1l5sl9:kvL_AjJptBPwzAfx8uTJLP1wrEl7k5aHvnpB2Xbt9bE/

Question
Is this a settings issue on my end or something else deeper on the allauth end? I'm using a settings.py file for development, and a settings-production.py file for production, and just calling any commands with a --settings=settings-production.py flag.

This should be on your end. I think what is happening is your emails are calling the Site in your db. In your admin, Sites> Site and see what the URL is set there. My gut reaction is that is set to localhost:8080


Remove tar dependency

Looks like this one will be slightly more complicated because it has to select between extracting .tar and .zip depending on filename, yeah?
Also I am thinking that tar and zip can go under the auto_download feature as well.

@nathanbabcock Yaa, according to my observation. These are the possible formats we get .zip, .tar and .7z . For all the different platforms combined.
Tried to find a library, that could do the job for all these formats. But couldn't find one.
I still think it's possible using cfg os = <some-platform> for different libraries, different platforms and file formats.
My concern is ffmpeg-sidecar claims itself to be with minimal dependency. This feature might change that claim 😂

@nathanbabcock what do you say ? Should I move forward adding different libraries for different file archive formats.
Or maybe guide me, if I am getting the technical details wrong ...

Yes, I know what you mean. But I think in this case it is okay. The core of the library can be very lightweight but this download module can pull in some proper deps.
The ones you've chosen look good. I think we can skip supporting 7zip. I haven't checked in a while but I don't recall any of the sources providing a .7z (please correct me if I'm wrong on that)

@nathanbabcock You're saying that we pull dependencies on runtime for decompressing files ? Okay, let me look into that ..
This url seem to provide the .7z file : https://evermeet.cx/ffmpeg/getrelease

@VirtualPirate Great catch, you're right. 7zip is used for the Mac M1 binaries. I don't think the tar command can handle those properly, it is probably silently failing. I'm surprised there have been no bug reports about it. All the more reason to handle these with native Rust code.
Luckily I found a link on that website that will return a zip instead: https://evermeet.cx/ffmpeg/getrelease/zip (Source: https://evermeet.cx/ffmpeg/#api-download). So we only need to support  zip for Windows and Mac, and tar for Linux.
To clarify — I don't mean pulling dependencies at runtime. They can be regular dependencies just like you have them in this PR. I like your idea about conditional compilation to choose the right library for each platform. Let's do that and combine it with Platform-specific dependencies so that even during cargo build no time is wasted on unnecessary archive formats.

@nathanbabcock got it .. working on it ..


Setting to change the server folder's name

I'm suggesting to add a setting to change the name that will be used for the folder that will contain the server data

runDirectory.set(layout.projectDirectory.dir("runServer"))


Different Results between Mac and Ubuntu16.04

Hello i have tested on my mac and ubuntu server. it seems like that hava some bugs in ubuntu.
First:
In ubuntu,when i test domain name . no domain name results founds only ip results.

but mac seems well

Two:
In ubuntu, some results always timeout even -http-timeout=10000 set.

but mac seems well

i want to use aquatone on my ubuntu server.please correct it.

ubuntu server has some problem.


Add Header Indicating Suspense Cache HIT

fixes #786

@james-elicx could you have a look at the e2e failure? :sweat_smile:  https://github.com/cloudflare/next-on-pages/actions/runs/9170973019/job/25220047035?pr=787#step:4:679
it does seem legit :sweat:

@james-elicx could you have a look at the e2e failure? 😅 cloudflare/next-on-pages/actions/runs/9170973019/job/25220047035?pr=787#step:4:679
it does seem legit 😓

can we delete the e2es? smh


The EngineVersionCheck.shouldUpdate() deletes the value of the CURRENT_ENGINE_RELEASE

When the value of the CURRENT_ENGINE_RELEASE property is the same as the value of the updateToVersion the method updates the property with the empty string value.
This seems no sense, since this property is used to compare against the last release readed from web.
The problem is that the value is overwritten with an empty string, because when no new version is available the code is executed one of twice.

When properties does not match then CURRENT_ENGINE_RELEASE property is updated.
Next time the properties match, so the CURRENT_ENGINE_RELEASE property is overwritten with the empty string (I think has no sense).
Next time properties does not match, so CURRENT_ENGINE_RELEASE property is updated.

In OracleDB there is no difference between empty strings and null: it stores a null string. After that when the plugin read the records in properties table and stores its values in HashMap it fails with a NullPointerException.

This should now be resolved in the current snapshot.


How do I enable the gcc maker.

 I've tried multiple ways. I have installed neomake with https://github.com/junegunn/vim-plug if that matters.
Expected behavior
gcc maker should be enabled
Steps to reproduce
Put one of the following lines in the vim config:
let g:neomake_cpp_enabled_makers = ['gcc']
let g:neomake_enabled_makers = ['gcc']

Output of the ":verb NeomakeInfo" command
Neomake debug information
Async support: 1
Current filetype: cpp
Enabled makers
For the current filetype (with :Neomake):

gcc
errorformat: '%-G%f:%s:,%-G%f:%l: %#error: %#(Each undeclared identifier is reported only%.%#,%-G%f:%l: %#error: %#for each function it appears%.%#,%-GIn file included%.%#,%-G %#from %f:%l,,%f:%l:%c: %trror: %m,%f:%l:%c: %tarning: %m,%I%f:%l:%c: note: %m,%f:%l:%c: %m,%f:%l: %trror: %m,%f:%l: %tarning: %m,%I%f:%l:
note: %m,%f:%l: %m'
args: ['-fsyntax-only', '-Wall', '-Wextra']
get_argv: function('2', {'get_argv': function('2')})
exe: 'g++'
NOTE: you can define g:neomake_cpp_enabled_makers to configure it (or b:neomake_cpp_enabled_makers).

For the project (with :Neomake!): None.
NOTE: you can define g:neomake_enabled_makers to configure it.
Default maker settings:

remove_invalid_entries: 0
buffer_output: 1

Settings
g:neomake_place_signs = 1
g:neomake_c_enabled_makers = ['gcc']
g:neomake_cpp_enabled_makers = ['gcc']

shell: /bin/bash
shellcmdflag: -c
Windows: 0

:version
NVIM 0.2.0-dev
Build type: RelWithDebInfo
Compilation: /usr/bin/cc -Wconversion -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DDISABLE_LOG -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wvla -fstack-protector-strong -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -DHAVE_CONFIG_H -D_GNU_SOURCE -I/builddir/build/BUILD/neovim-0
.1.7-2a366b6/build/config -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/src -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/.deps/usr/include -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/.deps/usr/include -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/.deps/usr/include -I/usr/include -I/usr/include -I/usr/include -I/usr/in
clude -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/build/src/nvim/auto -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/build/include
Compiled by mockbuild

Optional features included (+) or not (-): +acl   +iconv    +jemalloc +tui
For differences from Vim, see :help vim-differences

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

:messages
"git/epics-modules/m-epics-ifcfastint/src/IFCFastIntDIChannel.cpp" 465L, 17791C
NVIM 0.2.0-dev
Build type: RelWithDebInfo
Compilation: /usr/bin/cc -Wconversion -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DDISABLE_LOG -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wvla -fstack-protector-strong -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -DHAVE_CONFIG_H -D_GNU_SOURCE -I/builddir/build/BUILD/neovim-0
.1.7-2a366b6/build/config -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/src -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/.deps/usr/include -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/.deps/usr/include -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/.deps/usr/include -I/usr/include -I/usr/include -I/usr/include -I/usr/in
clude -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/build/src/nvim/auto -I/builddir/build/BUILD/neovim-0.1.7-2a366b6/build/include

Related question: How do I add arguments to gcc so that it finds my includes?

It seems that it was enabled all along, but it probably didn't parse the error. I got a fatal error because gcc couldn't find a header file.
Then I used let g:neomake_cpp_gcc_args = [... to add my include directories.

I got a fatal error because gcc couldn't find a header file.

The default error format should have matched this, so it can be considered a bug I think.

@NickeZ
Can you post the output from when not using the proper args, i.e. the fatal error, please?

@blueyed Sorry, I was probably just confused. By default neomake doesn't open the list of issues and I remember that I looked for some marker in the gutter but I couldn't see one. I can't reproduce the issue any more...

It would have been nice if the readme mentions g:neomake_open_list and that some documentation about the C maker mentions how to set include paths: let g:neomake_c_gcc_args = ['-fsyntax-only', '-Wall', '-Wextra', '-I/arg/to/include']. I looked everywhere and found the solution in some issue..


Error maximum recursion depth exceeded in comparison occurred while running config file _row_validation.yaml. Skipping it for now.

Row validation for more than 140 columns is failing with the error
Error maximum recursion depth exceeded in comparison occurred while running config file products_row_validation.yaml. Skipping it for now.
Details  -
Source : BigQuery External Table
Target : BigQuery Native Partitioned Table
Primary Key : Concatenated primary key using custom query ( as multiple primary key was not working )
Category : Random records and Filtered records validations
Please refer the below details more information

column count
record count
category
status
time taken
id
comments

140
1000
Random
success
~7 mins
452e8c66-cff5-4948-ad1a-090bb4ed89cc

28
1000
Random
success
~1 mins
2ec15483-18ce-40dc-ae86-0685599df82d

150
1000
Random
failure
nA
nA
Error maximum recursion depth exceeded in comparison occurred while running config file products_row_validation.yaml. Skipping it for now

145
1000
Random
failure
nA
nA
Error maximum recursion depth exceeded in comparison occurred while running config file products_row_validation.yaml. Skipping it for now

141
1000
Random
failure
nA
nA
Error maximum recursion depth exceeded in comparison occurred while running config file products_row_validation.yaml. Skipping it for now

168
4740
Filtered
failure
nA
nA
Filter - Error - Error maximum recursion depth exceeded in comparison occurred while running config file products_row_validation.yaml. Skipping it for now

168
1
Filtered
failure
nA
nA
Filter - Error - Error maximum recursion depth exceeded in comparison occurred while running config file products_row_validation.yaml. Skipping it for now

Please refer the YAML :
`result_handler:
project_id:
table_id: <dataset.table>
type: BigQuery
source: BQ_CONN
target: BQ_CONN
validations:

calculated_fields: []
comparison_fields:

cast: null
field_alias: col1
source_column: col1
target_column: col1
<OTHER 168 COLUMNS >
custom_query_type: row
filter_status: null
filters: []
format: table
labels:
!!python/tuple

validation_type
row
primary_keys:

cast: null
field_alias: pk
source_column: pk
target_column: pk
random_row_batch_size: '1000'
schema_name: null
source_query: "select pk1 || pk2 || pk3 || pk4 as pk, <other 169 columns>
\ \nfrom <dataset.external_table>
\ \nwhere date = "2023-07-01""
table_name: null
target_query: "select pk1 || pk2 || pk3 || pk4 as pk, <other 169 columns>
\  \nfrom <dataset.table> \nwhere
\ date = "2023-07-01""
target_schema_name: null
target_table_name: null
threshold: 0.0
type: Custom-query
use_random_rows: true`

Kindly let me know if you need more details on this.
Thanks

I suspect the upper limit may differ based on machine type/size of the columns. I hit the recursion error with ~98 columns. After the fix in the branch, I can validate up to ~140 columns. The fix here will increase the upper limit, but not completely remove it.
The workaround here is to split the validation into two commands - one for the first 70 columns, another for the second 70 columns for example.
Once it's merged, would be curious to see how many columns you can validate with the same YAML.


[Fluent] Porting and passing two unit tests + some bug fixes

Hi @martinsawicki, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (marcins). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@martinsawicki please resolve merge conflicts.

hmm, not sure why that happens - i just do the same thing as usual.
i'll look into this when i'm finished adding the rest of the test work soon.

@martinsawicki here is the sequence:

git pull upstream Fluent
git reset --soft upstream/Fluent
git commit -m "name for your commit"
git push origin Fluent -f

However for safety I'd suggest you to create a new brunch form your origin/Fluent and then create a pr from that new branch and close this one.


Update examples

Update examples where possible for usage of server option.
Excluded https as it takes options for ssl.

ops


The training loss of the affine registration network keeps going up

Thank you so much for sharing your outstanding work, which has helped me a lot. But when I use my own data to train the affine registration network, the training loss keeps going up. I have checked my training input and there is no problem. I set batch size to 1 and the learning rate to 0.00001. Do you know what the problem might be?

Hi @XiaotianJia ,
I've tried the affine network again for 8 epochs and it seemed to be working fine:

There were a few typos in the script that I made when sorting the code; I've corrected them now and they should have no effect on performance or loss while training. One suggestion I would offer is to experiment with other loss functions, depending on the modality you are working on, but I've found that MIND loss works the best.
Junyu

Hi, thank you for the great work. Somehow, I also have the same problem on the OASIS dataset using TransMorph default config.
The loss is increasing and val DSC score is around 0.6. I'm using Pytorch 1.10.1 with cuda 11.

@junyuchen245 Thanks for your reply. It is the deformable model. I used the scripts and the OASIS data you provided.
I just tried PyTorch 1.9, and it works now. I guess it is the PyTorch version problem.
Thanks

@ywzcode Sounds good! Thanks for letting me know.
Junyu


Docker exec and Orphan Process problem

-->
Description

Steps to reproduce the issue:
1,
docker run -ti -d aa732f074453 bash
ca5ebc10df7df14ba706a606b00866613cc3e69d34c27375cb63ef4a36bbdb2e
docker exec -ti ca5ebc10df7df14ba706a606b0 bash
sleep 1000&
exit  (hangs util sleep finish）

2,
ps -elf
F S UID         PID   PPID  C PRI  NI ADDR SZ WCHAN  STIME TTY          TIME CMD
4 S root          1      0  0  80   0 -  2872 n_tty_ 17:26 ?        00:00:00 bash
4 S root         45      0  0  80   0 -  2872 wait   17:29 ?        00:00:00 bash
0 S root         66     45  0  80   0 -  1026 hrtime 17:29 ?        00:00:00 sleep 1000
0 R root         67     45  0  80   0 -  3338 -      17:29 ?        00:00:00 ps -elf

the sleep process PPID is the bash we run，but after  we exit the bash，and PPID becomes 0
ps -elf
F S UID         PID   PPID  C PRI  NI ADDR SZ WCHAN  STIME TTY          TIME CMD
4 S root          1      0  0  80   0 -  2872 n_tty_ 17:26 ?        00:00:00 bash
0 S root         66      0  0  80   0 -  1026 hrtime 17:29 ?        00:00:00 sleep 1000
4 S root         76      0  0  80   0 -  2872 wait   17:32 ?        00:00:00 bash
0 R root         96     76  0  80   0 -  3338 -      17:32 ?        00:00:00 ps -elf

if we try outside the container ,we will find out the ppid is actually the docker-containerd-shim process
Describe the results you received:
1， exit hangs
2， ppid become 0
Describe the results you expected:
1， exit quit normally
2,  ppid become 1
Additional information you deem important (e.g. issue happens only occasionally):
always
Output of docker version:
Client:
 Version:      1.12.2
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   bb80604
 Built:        Tue Oct 11 17:00:50 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.2
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   bb80604
 Built:        Tue Oct 11 17:00:50 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 12
 Running: 11
 Paused: 0
 Stopped: 1
Images: 10
Server Version: 1.12.2
Storage Driver: aufs
 Root Dir: /data/docker/aufs
 Backing Filesystem: xfs
 Dirs: 163
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: host null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options:
Kernel Version: 3.10.104.xxxxxx
Operating System: <unknown>
OSType: linux
Architecture: x86_64
CPUs: 48
Total Memory: 125.3 GiB
Name: xxxxxxxxxxxxxxxxx
ID: VTSE:3AJP:MWYG:XSRO:VLHA:7RUN:S5KP:SDB4:HVDP:5HT3:24XC:VW52
Docker Root Dir: /data/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Insecure Registries:
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
centos 6 with 3.10.x kernel version.

ping @mlaventure @justincormack is this similar to https://github.com/docker/docker/issues/25348 ?

@thaJeztah I think the number 2 problem is similar to 25348 ,and I will try Runc rc4 to test , but I am not sure about the first on ,why exit will hangs until child process exits.

@thaJeztah  https://github.com/opencontainers/runc/pull/1018  It seems that  --console flag is removed and add --console-socket . So when running the new version of runc with docker ,  an error was reported : docker: Error response from daemon: oci runtime error: flag provided but not defined: -console.

Issue 1 is the expected behavior
Issue 2 should be fixed from 1.12.3 onward if I'm not mistaken.
Please update to 1.12.5 to fix the later.
To avoid the former ensure you close the new background process IOs, for instance :
sleep 1000 0<&- 2>&- 1>&- &

@mlaventure  1， it is not a excepted behavior , I don't have the problem when using docker 1.6.2
issue  2 seems not fixed, it 's similar to #25348 ,and it's a runc problem.

@sandyskies

is the correct behavior, we are waiting for the other side to close is IOs before exiting to avoid losing any logs. Since the sleep process keeps them open, we are not returning.

is working fine for me on 1.12.5 and 1.13.0. #25348 only applies when sharing a PID namespace with another container of the host

I will close this one, since runc is working on it .

https://github.com/opencontainers/runc/pull/993 @mlaventure  This fixes my issue，I will close.

not util https://github.com/opencontainers/runc/pull/976 is fixed

https://github.com/containerd/containerd/pull/1430 I create a pr for this issue,but just for 1.12.x ,hope someone find it useful.

Closing in favor of https://github.com/moby/moby/issues/34683


This extension does not load the logs from the dotnet upgrade-assistant

Hi, I am in the process of upgrading a project to .NET 6.0. I use the dotnet upgrade-assistant .
This assistant produces logs in the sarif-format. But they cannot be opened with this VSC-extension.

'c:...\AnalysisReport.sarif' was not loaded. Version '2.1.0' and schema 'http://json.schemastore.org/sarif-2.1.0' is not supported.

Is there a plan to support this version 2.1.0?

@klacol We support schemas ending in 2.1.0-rtm.5. These represent the current (most common) release. Confusingly, the suffix-less 2.1.0 was considered pre-release due to a versioning mishap.
It seems dotnet upgrade-assistant is emitting the wrong schema. I believe this was addressed with a recent PR.

@jeffersonking  Thanks for your fedback.
I have changed the version string in the sarif file, that is generated by the dotnet upgrade-assistant from 2.1.0 to 2.1.0-rtm.5:

Unfortunately, the sarif-vscode-extension does not load this file.

I am working with a newer version of the assistant, as of this tip.

@klacol I realize it's not intuitive, but the version should stay as 2.1.0. Altogether:
"$schema": "https://schemastore.azurewebsites.net/schemas/json/sarif-2.1.0-rtm.5.json",
"version": "2.1.0",

Thanks, that does not work, That was my starting point.

Trying to see what I'm missing here.
In your screenshot:
"$schema": "https://schemastore.azurewebsites.net/schemas/json/sarif-2.1.0-rtm.5.json",
"version": "2.1.0-rtm.5",

In my snippet:
"$schema": "https://schemastore.azurewebsites.net/schemas/json/sarif-2.1.0-rtm.5.json",
"version": "2.1.0",

Does the latter $schema + version not work for you?


Add intro to index of Hub docs

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Is it ok for me to merge?


webpack 3.3 will cause some error

Are you submitting a bug report or a feature request?

What is the current behavior?

I use How to reduce redux-form's bundle size?  to reduce the size of redux-form.
6.x work good, but when i update to 7.x, i modify anything in the project, have to restart the webpack watch process, or else, there is Uncaught TypeError: (0 , _reduxForm2.default) is not a function.
What is the expected behavior?
Sandbox Link

What's your environment?

node: latest
npm: latest
webpack: latest
babel: latest
redux-form: latest
Other information

Hmm... your node_modules/redux-form/es/reduxForm.js does have an export default in it, correct?

@erikras thanks for your reply. that maybe a webpack 3.3.x bug.
rollback to webpack 3.0.0 it work well.

@geminiyellow Are you using webpack 3 moduleconcatenationplugin? Theres a bug in that that was resolved in 3.4.0 that broke default exports in many modules.

@pbarbiero thank you,
yes, i using moduleconcatenationplugin and i has been tried 3.4, same error.

I am having this issue as well. Last version that worked for me is 3.1.0.

@spencerbeggs
<EMAIL_ADDRESS>+ redux-form@latest is worked for me.
wating webpack team fix the bug.

@geminiyellow Can you identify what the issue is? I thought it might be in the transform.

here
webpack --watch breaking site after modifying a file #5399

Closing due to inactivity.


Select series to view

This happens regularly but not always. I can't see when it does and when it doesn't. I select a series, and either double click or go to Vieww Series/Image in the menu. Then it asks me to select a series. See screenshot

Duplicate of #97.
I saw this issue after running merge script, but it's not because of merge only.
This is a tricky one because there seem to be multiple causes.

I'm moving this to the first milestone. I found where the real cause is and I'm adding Steve to help solve this.
I spotted this while resolving issue #94.
What's happening is that if a Series has "Series" in its id/description then this will be removed when being read in TreeView and weaselXMLReader. This is because we have bits of code where we use the function replace "Study" and/or "Series" with "". This is similar to the Image display title we had a couple of weeks ago. Instead of replacing "Series" and then "-", we should be replacing "Series - " @IMI-TRISTAN

Made the above changes to replace functions to extract subject, study, series and image id from the labels in the tree view.

Thank you for doing this change, I will test this very shortly and close if I'm happy with it.

It's all good @IMI-TRISTAN , can I make a quick change to TreeView.py or are you using it?

Yes, I have finished with TreeView.py
On Mon, 22 Mar 2021 at 13:26, João Almeida e Sousa @.***>
wrote:

It's all good @IMI-TRISTAN https://github.com/IMI-TRISTAN , can I make
a quick change to TreeView.py or are you using it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/QIB-Sheffield/WEASEL/issues/96#issuecomment-804059332,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKUTUZD4IXMB5E72OGLHGJ3TE5APTANCNFSM4ZFNYPPA
.

Perfect, I just committed this change and it's all working well


No error in analyzer for super.call

According to the https://github.com/dart-lang/language/issues/734#issuecomment-565487654 calling a call method via super call is an error for now. But analyzer still accepts the following code
class A {
  String call() => "A.call!";
}

class B extends A {
  String foo() => super();  // error in CFE, no error in analyzer
}

main() => print(B().foo());

Tested with dartanalyzer version 2.9.0-5.0.dev

Still no error in analyzer.

I think the rules changed; CFE now allows this as well. Likely this can be closed.

I don't remember the rules changing, but if it already works, maybe we can change them now.

https://github.com/dart-lang/language/issues/1631


Allow the team organizer to set the battletag per player

Currently you just see a field for the 'name' of each player, and that will be saved in the players.name field. To let the team organizer see hero confidence levels for their players, the players.battletag field has to be populated so we can link players to the users they represent.
I'm not sure how we'd work this into the UI. Maybe a menu for each player, possibly triggered in the players column of the composition form, with an option 'Set battletag'? Then if a battletag is set for a player, we could show it written below the player name in smaller text. Thoughts on this design @smbriones?
This issue needs:

A Rails endpoint to uniquely identify a specified player (maybe by its id or by its name + a new creator_id field on the players table that could contain the current_user who created that Player?) and set their battletag.
UI to hit this endpoint.
UI to display player battletags when they're set.

This was fixed by #102.


Confusing "Development status" (pre alpha) on pypi.org?

safety version: all
Python version: all
Operating System: all

Description
Pre-Alpha development status is displayed in pypi.org.
It is true?
If not, is it possible to correct the status?
It yes, do you plan to stabilize safety for use in a production environment? (status = stable)
What I Did
I viewed the page: https://pypi.org/project/safety/

it's because of this line https://github.com/pyupio/safety/blob/55cc69d40418424d4fdb776963dbf48ed2f7fa4f/setup.py#L45

@yeisonvargasf do you accept PR? I can have few proposal to improve this part (not only some flags)

Also if handling this, maybe  in setup.cfg also remove the universal build as only py3 is supported.
https://github.com/pyupio/safety/blob/55cc69d40418424d4fdb776963dbf48ed2f7fa4f/setup.cfg#L2
Right now the download file is named safety-1.10.3-py2.py3-none-any.whl

sorry late answer, PRs are welcome.

We have already fixed this issue. Thank you all.


Line break "Before single-line comment" does not ignore commented-out code (////)

I experience the problem below using StyleCop 2017.1.1 for ReSharper 2017.1.2.
In Resharper Options > Code Editing > C# > Formatting style > Blank Lines, I have "Before single-line comment" set to 1. This gives the following preview in the Options dialog:
void Foo()
{
    var x = 5;

    // comment
    Call();
    ////CommentedCall();
}

However, when I make a file with this code and format it using Reformat Code, this is the output I get:
void Foo()
{
    var x = 5;

    // comment
    Call();

    ////CommentedCall();
}

Notice that a blank line was inserted above ////CommentedCall();. This is contrary to the preview in the Options dialog. The commented call should be ignored and not get a blank line above.
I do not experience this problem after uninstalling the R# StyleCop plugin, even when using the exact same settings by adding the StyleCop settings file to the "My Computer" settings layer.

@cmeeren I can't reproduce this. Is it still actual? Can you check your settings once again by going to ReSharper | Options (not via Manage Options), which is the accumulated view of effective settings.


Typescript-only-props-typing feature is awesome, will it be released later?

What problem does this feature solve?
I found it on https://github.com/vuejs/rfcs/blob/function-apis/active-rfcs/0000-function-api.md#typescript-only-props-typing,
this is more typescript style than declare type again in props option.
Will it be released later or abandoned?
What does the proposed API look like?
just as  https://github.com/vuejs/rfcs/blob/function-apis/active-rfcs/0000-function-api.md

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.

It is implemented


Add Language: Go+

Description

Checklist:

[x] I am adding a new language.

[x] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?q=extension%3Agop&type=Code
https://github.com/search?q=extension%3Aspx&type=Code

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

Repository: uiuing/ingop
Case catalogue: uiuing/ingop/releases-migration

Sample license(s):

Apache License 2.0

[x] I have included a syntax highlighting grammar: goplus/vscode-goplus
[ ] I have updated the heuristics to distinguish my language from others using the same extension.

Other notes:

Go+ Language Repository

 The Go+ language supports the following file extensions: .gop .spx .gmx

More about the Go+ language in practice

https://github.com/goplus/spx
 https://github.com/goplus/AircraftWar
https://github.com/goplus/FlappyCalf
https://github.com/xushiwei/BetaGo
https://github.com/xushiwei/Dinosaur
https://github.com/sunqirui1987/HelloWorldForSpx
https://github.com/xushiwei/FireBullets
https://github.com/xushiwei/Gobang
https://github.com/goplus/MazePlay

I'm sorry if some of my mistakes have wasted your time

Closes https://github.com/github-linguist/linguist/issues/6045

Related issues: issues/6045

This is no where near popular enough for inclusion right now with only 774 files indexed across the whole of GitHub with most of these files owned by five users.

This is no where near popular enough for inclusion right now with only 774 files indexed across the whole of GitHub with most of these files owned by five users.

Thank you and I will be back when it meets the requirements. 😁

In addition to @Alhadis's suggestions, these samples are terrible and not real world samples… they're illustrations of usage, so will need to be replaced with real examples.

Sorry for the trouble and thanks for the advice! It's really important.

Closing as the requested changes have not been made in well over a year since being requested. Please open a new PR when you have the time to make the requested changes.


Cannot connect devices

Device not charging
I use USB A cable, it working in normal computer
Replug not working

Go into recovery mode first

This was fixed, redownload 1.0.8 from releases page.


DOC: Add --index-metadata to pybids layout faq

One source of #2538 may be a bad command in the FAQ. Without --index-metadata, the created database will not pull in RepetitionTime or any other metadata we require.

Codecov Report

Merging #2546 (358e1ad) into master (8b86156) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2546   +/-   ##
=======================================
  Coverage   44.83%   44.83%
=======================================
  Files          41       41
  Lines        3089     3089
=======================================
  Hits         1385     1385
  Misses       1704     1704

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 8b86156...358e1ad. Read the comment docs.


Failed blocks prop type

Versions
@sanity/block-content-to-react    2.0.6
react                             16.8.0-alpha.1

Issue
I've a pretty standard setup:
import BlockContent from "@sanity/block-content-to-react";

const Footer = ({ footerCards }) => (
  {footerCards.map(footerCard => (
    <div key={footerCard._id}>
      <FooterCard title={footerCard.title}>
        <BlockContent blocks={footerCard.information} />
      </FooterCard>
    </div>
  ))}
);

export default Footer;

But I receive a warning in the console:
Warning: Failed prop type: The prop `blocks` is marked as required in `SanityBlockContent`, but its value is `undefined`.

But everything else except this error seems to work fine and value of footerCard.information isn't undefined.

Sorry, found the problem, my bad.

I figured it out, I was using the wrong name of the field in my query so blocks was undefined. Thanks for responding tho.

Could you please elaborate more on this? Not sure if you still remember though.


Refresh button does not refresh data in

Please do not submit support requests or "How to" questions here. For that, go to StackOverflow.
What you were expecting:

Refresh button should refresh data in ReferenceManyField
What happened instead:

Data in ReferenceManyField is not refreshed (no API call) when Refresh button click
Steps to reproduce:

Add ReferenceManyField with reference="xyz" target="zyx", Datagrid, and TextField. Click on Refresh button.

I am facing the same problem. need help.

I am having similar issues.

Are you in an Edit or Show view?

In edit view

Confirmed, it's a bug. Looking for a fix.


Better Source Map Handling

Source map handling in various build functions is quite brittle right now.
Adding new build tasks that make source maps can sometimes be complex. Making sure those tools can also consume already made source maps is even worse.
Perhaps a source map function that could be told... remove these fields, fix these paths, etc... could be used by all build tasks instead of having said code spread all over the place, like it is now.

New code is going into the source-maps branch.

Better source map handling code is now live for everyone on GitHub and NPM. ^_^


fix(ios-google): allow to enable or disable zoom with double tap gesture after a MapView is created

Does any other open PR do the same thing?

No
What issue is this PR fixing?
#4522
How did you test this PR?

I created a test project with the minimum code I provided in the issue and verified the fix there.

Closing this PR as there's a more comprehensive one opened: https://github.com/react-native-maps/react-native-maps/pull/4532


[BUG] Nexus is down

Hi ! :)
The bot has been disconnected since this morning, I don't know if you are aware of it, but it was to warn you :)
Thanks

Just as I am writing the message it is back online ... Sorry for the useless issue

Hi @arthprin
Thank you for the info.
How long was it offline?

He was offline since 11am (UTC + 2) but is back online

Okay, thank you for the info.
In case it is offline later on - don't hesitate to reopen this issue.
I'm getting mail from the Github when someone creates/reopens/comments on the issue, so I will be able to manually restart bot quickly (most of the time)

I will also investigate this one, to make sure it is not a code-related issue I can fix.
But for now, I think it's something going on on the hosting side.
Sorry for the inconvenience.

Okay, no worries, thank you for your responsiveness :)


SAK-42396 Moved permissions webcomponent to sakai i18n

This commit makes the permissions web component fully internationalised,
using the same translation bundle as the permissions helper in
admin-tools. Translations for admin-tools have now gone into tomcat/lib
so the Sakai i18n webservice can find them. sakai-i18n.js uses that
webservice and the permissions and rubrics components use sakai-i18n.js.
https://jira.sakaiproject.org/browse/SAK-42396

@mpellicer Miguel, sorry I closed the PR earlier - I noticed that the permissions wc was not even picking up the exising i18n strings for the permissions helper. I've fixed that now.

Looking forward to having this implemented so we can use it as an archetype so all components can be i18n!

You'll just be able to extend SakaiElement, or import sakai-i18n.js and use that in your own base class.


Telegram bot: Use the bot instead of channel

Since @iammarco11 figured out that a telegram channel can't be flexible enough to serve out needs, my suggestion is to use the telegram bot itself, instead of public service from the bot as a how channel would have served, it would be a bit stricter option where users have to activate the bot in order to keep receiving the tweets.
The workflow idea:

We keep the twitter bot keep running, which scrapes all the tweets, tweet id etc which is needed.
Have a telegram bot, for instance, NaWab bot. When  /start is initiated at the bot, the bot can read/look at the most recent tweets scraped by the twitter bot and start sending it to the telegram bot. This can be done either by providing APIs from twitter scraping script we have, or either can read tid_store. An API to read tid_store would be the ideal way to proceed.
Add retweet inline keyboard option in telegram bot for admin specific use only.
We can customize and personalise the telegram bot as the above proceeds.

Let me know if there is any more questions.

Closed with #13


SEEED XIAO - timed out or error in transfer to remote: b'F'

As below, I seem to be unable to copy any files to the device.
Also, even setting the --buffer-size=128 gets ignored and the 512 USB default is used.
Files are created with zero bytes, I also tried the --ascii option but it made no difference.
Any suggestions ??
> rshell --buffer-size=128
Connecting to /dev/cu.usbmodem0000000000001 (buffer-size 512)...
Trying to connect to REPL  connected
Retrieving sysname ... unknown
Testing if sys.stdin.buffer exists ... N
Retrieving root directories ... /servo.py/ /bq2060.py/ /notes.txt/ /bqfunc.py/ /bqconfig.py/
Setting time ... Dec 24, 2021 14:44:58
Evaluating board_name ... pyboard
Retrieving time epoch ... Jan 01, 2000
Welcome to rshell. Use Control-D (or the exit command) to exit rshell.
> ls /pyboard -all
     0 Jan  1 2000  bq2060.py
     0 Jan  1 2000  bqconfig.py
     0 Jan  1 2000  bqfunc.py
     0 Jan  1 2000  notes.txt
     0 Jan  1 2000  servo.py
> cp flash/bq2060.py /pyboard
Copying '/Users/pac/docker/upydev/work/projects/aibo-xiao/flash/bq2060.py' to '/pyboard/bq2060.py' ...
timed out or error in transfer to remote: b'F'
>

I have the same problem with my esp32s2 (feathers2).
Running rshell on my Windows 11 Home PC, when I try "cp foo.py /pyboard/", it creates the directory entry for foo.py on the esp32, but it is zero bytes and the cp command fails with "timed out or error in transfer to remote: b''"

Same problem on Linux 5.15 with Raspberry Pi Pico and Qt-Py RP2040 (micropython 1.18), creates an empty file. Can't think of a solution, but for the meantime I started using mpfshell.

There was an issue with the --buffer-size that caused it to be ignored on the Rpi. I've just fixed that and it's available in version 0.0.31
That version also sets the default buffersize to be 32 for the RPi (the definition of an RPi in this context is a device with a USB VID:PID of 2e8a:0005)

So this can be closed?

Yes, it can be closed, thank you.
On Thu, Aug 25, 2022 at 5:56 AM Jose Riha @.***> wrote:

So this can be closed?
—
Reply to this email directly, view it on GitHub
https://github.com/dhylands/rshell/issues/173#issuecomment-1227102368,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEOHKJLRQISU7IIJSFBUNQ3V25GMZANCNFSM5KWEJE2Q
.
You are receiving this because you commented.Message ID:
@.***>


Refactor module layout

rename lib -> electroncash
move gui -> electroncash.gui
move plugins -> electroncash.plugins
fix all imports, remove module manipulations

Note that there is another pull request which changes module names: https://github.com/fyookball/electrum/pull/179
Electrum used lib, gui, and plugins modules and imported them as electrum, electrum_gui, and plugins respectively. Some config in the setup and packaging to handle it also. I'm not sure why it was done this way. Some possible reasons: historical, enable creating distributions without the gui or plugins code.
Anyway, I prefer this solution, I think it makes for a cleaner structure.

Note, this fixes #180

This PR will cause issues with other PR's currently submitted. Probably best to merge those if desired before this one. I'm happy to rebase this off a newer cash branch if needed.

im having a look.

Any reason some names went to electron and some to electroncash?

Closing this PR. I will rebase, reorganise, and resubmit as a new PR.

Reopening for visibility, but needs to be re-done


Token/key from a device missing

Hi,
how to get the token/key from a device?
Log entries:
[8/20/2024, 12:35:37 PM] [midea-platform] [net_ac_EA24 | <IP_ADDRESS>:6444}]
Cannot add new device <IP_ADDRESS>:6444, error:
Error: Token/key not provided in config file, cannot add new device
Discovery:
[8/20/2024, 12:35:37 PM] [midea-platform] Discovered device: {"ip":"<IP_ADDRESS>","port":6444,"id":SECRET"model":"00000Q12","sn":"SECRET","name":"net_ac_EA24","type":172,"version":3}
[8/20/2024, 12:35:37 PM] [midea-platform] Adding new accessory: net_ac_EA24

Run discovery in the plugin config.

Did already and I have also attached the discovery log here (see above)

Ah found it, within the plugin settings - thanks!

@dkerr64 mine was discovered but it says "token missing key missing". any advise?


Pin Kombu 3

Kombu 4.0.0 is not compatible with the version of celery frozen in requirements
  File "/opt/openduty/env/local/lib/python2.7/site-packages/celery/five.py", line 153, in <module>
    from kombu.utils.compat import OrderedDict  # noqa
ImportError: cannot import name OrderedDict

Thanks for the PR!


Transfer failure support

Adds support for the Transfer Failure endpoint and object

Thanks for the approval! Is there a reason you've not yet merged?

@tylerb No, just caught up in other things. Thanks for the contribution!


Feature/support mip: put boot.py and main.py under /lib/wifi_manager

Because installing using mip won't write outside of the /lib directory, the files /boot.py and /main.py must be put under /lib. The logical place to put these is under /lib/wifi_manager.
This PR adds these two files to the package.json and also adds documentation to the README.md file describing how to invoke this logic from the user's /main.py and /boot.py files.

I'll fix some other things and re-open this PR soon

Relates to https://github.com/brainelectronics/MyEVSE-Webinterface/issues/19


redis: add support for inline commands

Commit Message: redis: add support for inline commands
Additional Description: https://redis.io/docs/reference/protocol-spec/#inline-commands
Risk Level: Low
Testing: Unit tests added
Docs Changes: Added
Release Notes: Added
Platform Specific Features: None

I believe all review feedback has been addressed. Please review. 🙏

Looks like CI is failing and it need a main merge.
/wait

@weisisea can you take a look, thank you.
/wait-any

Needs to merge main.
/wait

@weisisea can you please take a look, thank you.
/wait-any

Sorry for the delay.. LGTM!

Can you merge main and we can get this merged? Thanks and sorry for the delay.
/wait

Done! Thanks and sorry for the delay on my end as well.

/retest


export and import settings option

hi , i want to suggestion / request an feature : export and import settings ,so we can save rule on old profile and export, then import to another chrome profile
Thanks

Hello @navmesh,
It's already available. See the screenshots in the README.
https://github.com/furybee/chrome-tab-modifier/raw/master/screenshots/settings.png
Best


Automatically detect ResponsiveGrid Width based on container width

Hello,
and thanks for this great library.
Looking at this example:
https://github.com/STRML/react-grid-layout/blob/master/test/examples/0-showcase.jsx
Seams like the grid is resizing smoothly while resizing the window.
I'd like to have the same result.
For now I can only set statically the width in one of the props of ResponsiveReactGridLayout.
I believe I can listen to windowResize and do the math, but would be a lot better if the width (when not expressed) just expand to the 100% of the div containing the GridLayout.
Thoughts?

Ok the catch is to use
import {Responsive, WidthProvider} from 'react-grid-layout';
const ResponsiveReactGridLayout = WidthProvider(Responsive);
to initialise the ResponsiveReactGridLayout.
I haven't dig into how costy it is vs having everything expressed in %, but for now it doesn't seams to be a bottleneck in my app.
Thanks again.


Question about this work.

Dear Andrey Savchenko,
I'm a student and going to build a small system to detect student's emotions for my thesis. After finding a solution, I found your job. But I can't run https://github.com/HSE-asavchenko/face-emotion-recognition/blob/main/src/affectnet/train_emotions.ipynb by current AFFECT dataset's version. Please correct me if I'm wrong. My question is: Can I run this workhttps://github.com/HSE-asavchenko/face-emotion-recognition/blob/main/src/affectnet/train_affectnet_march2021_pytorch.ipynb with MobileNet. Because I tend to build small applications to detect emotions from client site then send result to server.
Many thanks,
Son Nguyen.

Hello! I do not have the recent version of AffectNet, so I cannot modify the code by myself.
As far as I know, the images are the same as before, only their labels are organized in the csv file. Actually, that's was the reason to create the copy of my code for recent dataset that you mentioned. I believe you could easily convert this code for usage with my TensorFlow's implementation of MobileNet. Just load the labels of validation images that you need and match them with my predictions, but be sure about changes in position of labeling as it is written in train_affectnet_march2021_pytorch.ipynb. If you need to retrain the model, just change the imagedataloader and read labels from csv files with Pandas.

Hi,
Thanks for you reply, I had try create my own csv file by myself as your suggestion in previous closed issue, but it's seem to take a long time. I will try again like what you guide above.
Thank you!

Closing due to inactivity


disable webSecurity, add electron-debug, add semicolons

disables webSecurity to allow https resources to come through
added electron-debug which only runs in debug mode
add semicolons because jslint was yelling at me. then proceeded to disable jslint...

disabling webSecurity feels like a bad idea. see https://github.com/electron/electron/blob/master/docs/tutorial/security.md


Consider adding coinsanity/orbsanity YAML features

DLCQuest -> add feature for low coinsanity (< 5)
Jak -> add feature for low orbsanity (< 5?)
Mario Land 2 -> Also seem to have coinsanity that can get quite high (need to figure out a threshold)

That would be 5 features max, should also consider mobile users here and what to do on overflow.


Axis inconsistency

Closes #110

Context (#110)
Currently the Hex coordinate system follows the convention that y is going down, for flat hexagons mostly as pointy hexagons it would go down-right. This is inconsistent:

with the 2d Y-up convention of game engines like Bevy, this is not a big issue, as the hex coords don't relate directly to pixel coordinates
with the Direction enum, Direction::Top is equivalent of hex(0, -1) and Direction::Bottomhex(0, 1)` which is a major inconsistency with the coordinate system.

This PR provides a fix with 1e1103cb95fc8074f6c44a27b87e4aa737d82d36 which does the following:
Invert the y axis in HexLayout::hex_to_world_pos and HexLayout::world_pos_to_hex. This is a debatable breaking change
This Pr provides a fix with 47ce48105722ee582f0d850257401b2c658262e5 which does the following:
Adds the option to invert each axis in HexLayout, keeping the current behaviour as default to avoid breaking change
Other solutions:

[ ] Keep the y up as it is but rename Direction variants to correctly represent the orientation
[ ] Keep the y up but rename Direction variants based on axis instead of orientation
[x] Allow customization in HexLayout with something like y_up: bool which is probably a good idea

@uvizhe can you test this PR ? does it fix the #110 correctly ?

Tested it, orientation is good!
Side note: Bevy is always Y-down since 0.11.

Side note: Bevy is always Y-down since 0.11.

Yeah for screen coordinates but not for the world coords

@alice-i-cecile @uvizhe I think I prefer the new solution of 47ce48105722ee582f0d850257401b2c658262e5:
Instead of inverting the y axis in HexLayout I added the option to invert both, this is no longer a breaking change but will still require a minor version.
This means that the default behaviour is the same but I updated the hex_grid example to invert the y_axis. @uvizhe do you think that change fixes #110 ?
Should I invert the y axis by default still ?

I think Top is top and Bottom is bottom regardless of coordinate system. It's about how we see natural world and how describe it. For almost everybody (I guess 🙃), looking at computer screen, top and up are synonymous as well as bottom and down. So the default should be natural and there is no inconsistency and no room for misinterpretation here.
Having an option to invert axes is totally fine by me.

Since hexx is mostly used by bevy it makes sens to use its standard of y coordinates, therefore the y axis will be inverted by default


fix: update list components types to match RN and Flashlist

Currently, the list components (ListHeaderComponent, ListFooterComponent, and ItemSeparatorComponent) are typed as ReactNode. However, both React Native and FlashList support passing either a React component or a node.
This PR updates the types to have the same type definitions as React Native and FlashList.

Beautiful! Thank you ❤


Correct the default format for UUIDs

Correct the default format for UUIDs

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


SKE Credentials Rotation: complete-rotation command

Documentation follows the portal, shown here:

In general, getting some feedback from the SKE team regarding the commands help would be important

In general, getting some feedback from the SKE team regarding the commands help would be important

Good idea, but since this is a feature branch we could ask them after merging


Automatic cryo system

Short description

Sends SSD/Disconnected players in empty cryo storage unit. Don't work with died/crit players.
closes #154
Why we need to add this

No more banned afk players.
Checks

[x] I do not require assistance to complete the PR.
[x] Before posting/requesting review of a PR, I have verified that the changes work.
[x] I have added screenshots/videos of the changes, or this PR does not change in-game mechanics.
[x] I affirm that my changes are licensed under the Starlight Fork License and grant permission for use in this repository under its conditions.

Changelog

:cl: Rinary

add: Added automatic cryo system, now SSD/Disconnected player's sends in empty cryo storage after 5 minutes.

I'm so confused - what does this actually do?
Grammar is ambiguous

I'm so confused - what does this actually do?

And thank you for explaining 😊


pydantic.validate_call is converting strings to decimals when string is one of the allowed types

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
pydantic.validate_call is converting strings to decimals when string is one of the allowed types
No conversion should be done when an exact type match is listed in the allowed types.
So logic should be, see if exact type already listed in union, if so, return the value unaltered (no conversion/coercion)... otherwise do the heuristics that you normally do.
Not sure if this was fixed in https://github.com/pydantic/pydantic-core/pull/1332 . If so, was that code deployed to pypi?
The example code outputs
<class 'str'>
<class 'decimal.Decimal'>
10031853
10031853

Example Code
import decimal
import typing
import uuid
from datetime import date, datetime

import pydantic

@pydantic.validate_call(config=pydantic.ConfigDict(arbitrary_types_allowed=True))
def echo(value: date|datetime|float|decimal.Decimal|int|str|bool|None|typing.Callable|uuid.UUID) -> object:
    return value

expected = "10031853"
actual = echo(expected)
print(type(expected))
print(type(actual))
print(expected)
print(actual)

Python, Pydantic & OS Version
>python -c "import pydantic.version; print(pydantic.version.version_info())"
             pydantic version: 2.8.2
        pydantic-core version: 2.20.1
          pydantic-core build: profile=release pgo=true
                 install path: D:\code\venv\Lib\site-packages\pydantic
               python version: 3.12.1 (tags/v3.12.1:2305ca5, Dec  7 2023, 22:03:25) [MSC v.1937 64 bit (AMD64)]
                     platform: Windows-11-10.0.22631-SP0
             related packages: typing_extensions-4.12.2
                       commit: unknown

I also force install the latest version of pydantic-core... bug still there

>python -c "import pydantic.version; print(pydantic.version.version_info())"
             pydantic version: 2.8.2
        pydantic-core version: 2.21.0
          pydantic-core build: profile=release pgo=true
                 install path: D:\code\venv\Lib\site-packages\pydantic
               python version: 3.12.1 (tags/v3.12.1:2305ca5, Dec  7 2023, 22:03:25) [MSC v.1937 64 bit (AMD64)]
                     platform: Windows-11-10.0.22631-SP0
             related packages: typing_extensions-4.12.2
                       commit: unknown

It works when I make str the first thing in the union.
E.x.: def echo(value: str|date|datetime|float|decimal.Decimal|int|bool|None|typing.Callable|uuid.UUID) -> object:

Hi @hb2638,
If you use strict=True, you get the expected behavior:
import decimal
import typing
import uuid
from datetime import date, datetime

import pydantic

@pydantic.validate_call(config=pydantic.ConfigDict(arbitrary_types_allowed=True, strict=True))
def echo(value: date|datetime|float|decimal.Decimal|int|str|bool|None|typing.Callable|uuid.UUID) -> object:
    return value

expected = "10031853"
actual = echo(expected)
print(type(expected))
print(type(actual))
print(expected)
print(actual)

I think the real question here is why do smart unions select decimal.Decimal over str for str type data in lax mode, which has been raised here.
A more minimal reproducible example:
import decimal

from pydantic import TypeAdapter

expected = '10031853'

ta = TypeAdapter(decimal.Decimal | str)
actual = ta.validate_python(expected)

print(type(expected))
print(type(actual))
print(expected)
print(actual)

Going to close this as a duplicate.

thank you for looking into this!
I wish other projects were as on top of issues as this one.

Sure thing. I'm digging around for a fix now, hopefully will have one by tomorrow.

I've opened a PR with a fix, should be included in v2.9 by the end of the month!


Get osparc keys from config.ini

This is already handled here: https://github.com/nih-sparc/sparc.client/blob/main/src/sparc/client/services/o2sparc.py#L161


Fix breaking bug for certain webpack configurations

coffee-loader will cause the webpack compilation to crash with the following error:
Module build failed: SyntaxError: Unexpected token u
    at Object.parse (native)
    at Object.module.exports (workspace/node_modules/coffee-loader/index.js:39:18)

Upon inspection, the the result that this JSON.parse is attempting to parse shows a successfully-compiled coffeescript string/buffer, it does not have a v3SourceMap attribute, which makes JSON.parse attempt to parse undefined. I'm not doing anything especially unique with my webpack configuration except perhaps using the multi-compiler, I am not sourcemapping them, nor am I minifying.
Thank you for the great loader. I've enjoyed using it in so many of my projects.

Fixed the configuration to work with this loader as this repo is mostly inactive. RIP coffee-loader


Set the OTP apps for the PLT based on release version

This was determined by checking the contents of the lib directory for each of
the first release tags in a particular major OTP version, roughly like below:
for v in {17..24}; do
  git ls-tree -r -d "OTP-${v}.0" lib | \
    awk -F/ '{ print $2 }' | \
    sort -u > ../OTP-$v.txt;
done

for v in {17..23}; do
  diff -u ../OTP-$v.txt ../OTP-$((v+1)).txt;
  echo;
done

will double-check and run these checks too.


Bbox Strategy should allow for externalProjection

Strategy/BBOX is not currently allowing for the fact that the projection of the data source may differ from that of the layer. This commit checks for externalProjection on the protocol's format.
The checks that protocol/format exist are probably not necessary, but several tests fail if these checks are not in there. I added a dummy bounds in test_createFilter() to stop it complaining it didn't know what 'this.bounds' was.
There is not currently any test that the bounds are correctly set, so I've not added one to check that the externalProjection case works. However, it works on my own tests.

"The projection of the layer corresponds to the projection of the data source"
In which case, there's no need for externalProjection - it can be deprecated
"You should not use internal/externalProjection properties on a format when using strategies."
I have been using it with fixed strategy since it was created without any problem. It works well.
For the moment, rather than change all my code, which is based on the assumption that all layers are in the same projection, I can of course override the OL method with my own. However, if there are these limitations, they should be documented.

I'm also giving Fixed with externalProjection as an example in my little tutorial http://www.peterrobins.co.uk/it/olvectors.html
so I won't be the only one using it in this way :-)

The internalProjection and externalProjection properties on formats are useful when a format is used without a strategy.  I agree that this can be confusing or misleading.
Let's continue the discussion for sure - I didn't mean to completely close it off.  Are there things that you cannot do by setting the map projection and layer projection?

"Are there things that you cannot do . . ." - not sure, to be honest. I'd have to check :-)
"as with the BBOX strategy, you'll find that the Fixed strategy transforms geometries if both layer and map projection are set properly"
Yes, I know of this, but to me it produces false results. If I look, for example, at the vector layer in the wfs-reprojection example you mention, it tells me the projection is 4326. That would normally indicate to me that the data/features in that layer are in that projection, just as projection on a raster layer describes the projection of the tiles in that layer. But if I look at the features, they aren't in that projection at all. To me this is because in this case, the projection 4326 is not the layer projection, but the projection of the source data, which in this case is different. You could, I suppose, add externalProjection to vector layers and have the strategy use that, but then as the layer is linked to the protocol which is linked to the format which is what defines the data source and already has externalProjection, why duplicate this?

The layer projection corresponds to the projection of the source data.  This is akin to the externalProjection for a format.  They key point is that you shouldn't have to configure a format to specify the source projection of a layer.  This is the purpose of the layer projection.

at the moment, by default, layers are set to the map projection. A common usage is to display fixed-projection vectors, such as kml or gpx, on top of fixed-projection rasters, such as Google. If I'm using a fixed strategy to do this, I set my map projection to the raster projection, the externalProjection is defined in the format and, because the projections are not the same, the format transform the coordinates into the layer/map projection. This works perfectly well and, at least to me :-), seems logical. Why change this? If I've understood you correctly, you're saying that vector layers should not use the default, i.e. map, projection, but should be explicitly set by the developer to the externalProjection already defined on the format. Sorry, but I don't see the need for this. And I don't see why the same logic can't be used for bbox.

A related issue is that, for formats where externalProjection is not fixed, the format should be clever enough to fetch the crs from the file/data. There may be situations where the developer doesn't know what projection the file/data is in, for example, where the user defines at run time which file/data to load. internalProjection should, by default, be the map projection (I think we agree on that :-) ).


Enhancement : Take a ServiceDataSource as an arg, instead of Collection in the hubClient

Instead of taking the Collection in the AbstractRangerHubClient and building the serviceDataSource as follows.
 protected ServiceDataSource buildServiceDataSource() {
        return !services.isEmpty()
               ? new StaticDataSource(services)
               : getDataSource();
    }

Take the ServiceDataSource as an arg. In the above case, if someone intializes a hub without any service, then we end up getting everything from the respective datasource and there is no way for someone to init an empty hub.
While it may be reasonable to expect people to init a hub with services, in theory it may make sense to take in a serviceDataSource as the arg and default to the respective dataSource (http or zk) if service data source is null, in the case of an empty hub, then the clients can do a
new StaticDataSource(Set.of()) and the hub won't still scan the entire client's datasource.
@santanusinha : FYI if this makes sense?

Closing this, addressed in https://github.com/appform-io/ranger/pull/12


fix for seek bar issue for Kodi 19

I have added new array type params in jsonrpc request which is required by Kodi 19. Then used percentage instead of time for the seek bar.
This is more of a quick and dirty fix. I am not so much of an Android developer, so I am leaving this here for some else to pick up from there.

This implementation was following the JSON-RPC request created by Kodi' web remote, which apparently is not the official way of doing things. I will close this for now and create a new pull request with proper implementation


Honor :id assign

Currently the LiveComponent always has the id "some-unique-id". When the user passes in an :id assign, it should override the default value.

Fixed in 0.6.1 for LV 0.18

Fixed in 0.5.1 for LV 0.17


Generating tests for "all-your-base" exercise

Hi there!
I've been working on adding a test generator for the all-your-base exercise for exercism/xruby and I've run into a bit of a snag.
Everything works perfectly until I get to the last 13 (if I counted correctly) exercises that are left undefined. As far as I'm able to tell, and I certainly could be missing something, these tests aren't easy (possible?) to generate from individual tracks. Is that correct?
If so, I'm wondering if theres a way that we could add more information to the canonical-data so that different tracks could generate their tests based on the following (from the data documentation):
It's up to each track do decide:

1. What's the canonical representation of zero?
    - []?
    - [0]?

2. What representations of zero are allowed?
     - []?
     - [0]?
     - [0,0]?

3. Are leading zeroes allowed?

4. How should invalid input be handled?

Though, I'm really not to sure how that would work...
Have any other tracks successfully implemented a test generator for all-your-base? I looked through a couple of the other language repositories, but I wasn't able to find anything.
Thank you so much for your time!!

It's useful to link to a file you are discussing: https://github.com/exercism/x-common/blob/master/exercises/all-your-base/canonical-data.json

Aren't the last examples with expected: null error cases?
So you need to return/raise an appropriate error value.

@stevejb71 has created a test generated and converted all-your-base with it for the xocaml track.
He probably faces similar questions, so maybe he can weigh in on the matter?

@Insti

Aren't the last examples with expected: null error cases?

Not necessarily, for example the leading zeros test:
{
  "description"  : "leading zeros",
  "input_base"   : 7,
  "input_digits" : [0, 6, 0],
  "output_base"  : 10,
  "expected"     : null
}

This is a case where tracks can either implement leading zeros as valid or invalid.
So in the Ruby implementation, we do allow leading zeros:
def test_leading_zeros
    skip
    digits = [0, 6, 0]
    input_base = 7
    output_base = 10
    expected = [4, 2]

    ...

But the Javascript implementation does not:
xit('leading zeros', function () {
  expect(function () {
    converter.convert([0, 6, 0], 7, 10);
  }).toThrow(new Error('Input has wrong format'));
});

@dvberkel Thanks so much! I'll check out that implementation 😄

Hi @tommyschaefer
I also allowed leading zeroes in OCaml (it felt more natural, even though it complicates the implementation).
The OCaml test generator works off a template file (one for each exercise), and substitutes in a None (OCaml equivalent of null, to indicate failure) for the null it sees in the json. Since this isn't what I wanted, I just hand-edited.
I see the test generator as a tool to help in translating exercises into OCaml, and to help updating tests when the json changes, but it will still need some human intervention for certain exercises.

Thank you so much for the help @Insti, @dvberkel, @stevejb71!
I'm going to go ahead and close this issue since, with help from you all, I was able to get this working 😄
Thanks again everyone!!


fix: Fix completions not always working in for-loop patterns

Fixes https://github.com/rust-analyzer/rust-analyzer/issues/11205
bors r+

bors retry

bors r+


Adding Winning Line in Tic Tac Toe Game

fixed issue : - #465
screenShort  : -

Please merged this pull request @iamrahulmahato.


Develop

Mostly bug fixes and additional tests. Also adds a conf file and an example of using it to examples included in the distro.

Codecov Report

Merging #34 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #34   +/-   ##
=======================================
  Coverage   89.61%   89.61%
=======================================
  Files          22       22
  Lines         337      337
  Branches        4        4
=======================================
  Hits          302      302
  Misses         35       35

Impacted Files
Coverage Δ

...rkbench/datageneration/DataGenerationKickoff.scala
83.33% <ø> (ø)
:arrow_up:

...a/com/ibm/sparktc/sparkbench/cli/ScallopArgs.scala
100% <100%> (ø)
:arrow_up:

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 911b620...5b594bc. Read the comment docs.


Is Airpal deprecated?

Since there is not being developed in this repo...
Wanted to ask if the project is no longer used...

i'm have the same issues.

i'm have the same issues.

Last commit is from 2016 so you can assume so..

@ChethanUK Are you able to run the airpal application locally ? I am getting stuck after the first few steps and I am unable to move past them. Inspite of installing gradle and setting the appropriate system environment variables.

No, we had moved to quix instead


EMR cluster  "ModuleNotFoundError: No module named 'delegator'"

I can't get my script to recognize that delegator.py has been installed.  I'm running conda on the EMR cluster.  This are the lines from my bootstrap.sh file:
MINICONDA_PACKAGE=miniconda.tgz
tar zxf ${MINICONDA_PACKAGE}
/opt/miniconda/bin/pip install delegator.py
But when I run my script which starts with
import delegator
Using this command to run the script
PYSPARK_PYTHON=/opt/miniconda/bin/python python3  from_athena.py
I get the error described in the subject line.
Any ideas on how to get it to inderstand that it is already installed?

My invocation command was wrong.  This works:
/opt/miniconda/bin/python  from_athena.py
Apologies for wasting your time.


Mousewheel vertically not working correctly when scroll by touchpad

Swiper Version: Latest
Platform/Target and Browser Versions: Chrome/Safari

What You Did
scroll by touchpad with mousewheel feature
Expected Behavior
Working well as mouse scrolling
Link test: https://swiperjs.com/demos/270-mousewheel-control.html

not possible as it is impossible to detect when we touch/release touchpad


PortMaster GUI: Shuffled button layout on Powkiddy X55

Device Type

[ ] RG351P/M
[ ] RG351V
[ ] RG351MP
[ ] RG353M/V/P
[ ] RG503
[ ] RG552
[ ] RGB10
[ ] RGB30
[ ] RK2023
[x] X55
[ ] Other device (device name here)
[ ] All of the above

Firmware

[ ] ArkOS
[ ] ArkOS Wummle
[ ] AmberELEC
[ ] JelOS
[ ] uOS
[ ] RetroOZ
[ ] TheRA
[x] ROCKNIX (20240612)

GUI Track

[x] Release
[ ] Beta

Description

The UI indicates that A for confirmaion and B for cancel.

But the actual inputs are inverted so I have to press B to confirm and A to back. (X is properly mapped to X.)
It's confusing.
Reproduction Steps

Boot the device.
Open tools > PortMaster.
-> PortMaster-GUI opens.
Press A.
-> It asks you "Are you sure you want to exit PortMaster? A: Okay B: Cancel"
Press B.
-> PortMaster-GUI quits.

This is a bug In the OS, it worked before. Make sure to start PM from tools. In doubt delete the PM folder and start it again from tools.


Training on Multi-GPUs

I am pretty new to tensorflow although I have some experience with Keras. If I have two GPUs in my system, would it be possible to accelerate the training? If so, how? Apologies if this is a obvious question.

Yes I think so. But it's quite dirty to support multiple GPUs and I don't have a server with such setting so it's hard to implement it :( But there are some good references for it like:

https://github.com/tensorflow/models/tree/master/inception
https://github.com/openai/pixel-cnn

Not, a exact solution, but a temp workaround -- which might be obvious to a lot of folks, but still jotting it down here in case it helps anyone.
Instead of using multiple GPUs for one training, I went another route... I invoked one session per GPU like this.
CUDA_VISIBLE_DEVICES=0 python main.py --dataset=Shoes --use_gpu=True
CUDA_VISIBLE_DEVICES=1 python main.py --dataset=CelebA --use_gpu=True


Chatbot: chat initiation

Acceptance criteria
Other stuff (Nemo, Buddy)

The chat with response is initiated when:

User clicks a button on the chat landing page (Can’t we just put the AI in a box?)
User types at least one character and presses enter or the plane icon
User clicks one of the buttons from the widget on the home page

Opens in same tab

CSS (Melissa, Archana)
Links
https://www.figma.com/file/qqfeYAh6Z7QdyJXsQjIUfh/Stampy?type=design&node-id=1156%3A19240&mode=design&t=dg1esxPyfTqHQMnF-1

Update: there are still some styling issues

The sticky bottom bar. The shadow on the box.
Also, what the user actually types is not what is input, but rather the question above. This is okay maybe, but if we do it this way we need to make sure the question above is what is filled in on the chatbot as what the user said.

can you explain what you mean with the input?

Looks like it's fixed!

I'm assigning this to Myles because I think all the remaining fixes are in a pr


Add ERC20 increase_allowance - decrease_allowance + tests

Is your feature request related to a problem? Please describe.
Both fn missing from implementation
Describe the solution you'd like
Simply add implementation and tests
Additional context
depends on #761

I want to have a try. Have you been doing on this now? @ametel01

@0xAsten waiting for #761 to be merged


Feature/ng2 ui auth update

The new satellizer object with ng2-ui-auth is different from before, need to extract the code and redirect uri differently.
For issue #1505

Coverage decreased (-21.5%) to 42.114% when pulling 7a01787727029d6cbaf0bfd07695cd536b743b06 on feature/ng2-ui-authUpdate into 26acdd49ee5a4c80f426114aaab4b023eba7f931 on develop.

Closing for now until swagger codegen 2.4.0 is stable.


Working together with original Windows update

After instead LegacyUpdate,I still want to try original Windows update.I opened the ProxHTTPSProxy and the icon in the backup folder ,however the link still went to LegacyUpdate.net and can't get the original wu address .So could you fix the conflict ?

You’ll need to delete HKEY_LOCAL_MACHINE\Software\Microsoft\Windows\CurrentVersion\WindowsUpdate\URL, this overrides the url wupdmgr.exe (the Windows Update shortcut) opens to. You can also directly go to update.microsoft.com in IE.

You’ll need to delete HKEY_LOCAL_MACHINE\Software\Microsoft\Windows\CurrentVersion\WindowsUpdate\URL, this overrides the url wupdmgr.exe (the Windows Update shortcut) opens to. You can also directly go to update.microsoft.com in IE.

The website could open after edit rigedit but can't work with error 0x80096010.