Run ecto migration with non-standard path of repo

Environment

Elixir version (elixir -v): 1.2.4
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): pg 9.5
Ecto version (mix deps): 1.1.7
Operating system: ubuntu 15.04

I'm not sure it's a bug or my missunderstanding of ecto (but it looks like a bug), so I placed issue here and a question at SO. You can close this issue if you think it isn't a bug.
I have a "dummy" app in my mix app, so, it's first migration is placed in test/dummy/priv/repo/migrations/20160323060044_create_user.exs.
I want to run this migration, but it returns 11:19:20.647 [info]  Already up, and this disappoints me.
Here is my test.exs:
config :ecto_state_machine, Dummy.Repo,
  adapter: Ecto.Adapters.Postgres,
  username: System.get_env("USER"),
  password: "posgtres",
  database: "ecto_state_machine_test",
  pool_size: 10,
  port: 5432,
  priv: "test/dummy/priv/repo/"

Here is my migration:
defmodule Dummy.Repo.Migrations.CreateUser do
  use Ecto.Migration

  def change do
    create table(:users) do
      add :state, :string, null: false
      add :confirmed_at, :datetime

      timestamps
    end
  end
end

I'm running
MIX_ENV=test mix ecto.migrate -r Dummy.Repo

and get annoying already up instead of migration. What's I'm doing wrong?
My attempts:

I tried to drop and re-create database, nothing happened

My another turn was to require the code. I tried to implement this in my test.exs or dev.exs, but I received `module Ecto.Migration is not loaded and could not be found
"test/dummy/priv/repo/migrations"  |> Path.join("**/*.exs")  |> Path.wildcard  |> Enum.map(&Code.require_file/1)

I also tried to create a fake migration by MIX_ENV=test mix ecto.gen.migration f -r Dummy.Repo, fill it with code which raise an exception, and than I had
MIX_ENV=test mix ecto.migrate -r Dummy.Repo # => 12:49:00.932 [info]  Already up

I also tried to load shell MIX_ENV=test, then invoke code from 2nd approach, then Mix.Task.run "ecto.migrate" ["-r", "Dummy.Repo"]. it returned :noop to me?

Is it a bug or I missed something? Alex

This is fixed in Ecto 2.0.0-rc. In previous versions, migrations would always have to be inside priv. We have removed this requirement in Ecto 2.0.

@josevalim I updated ecto up to 2.0.0-rc5. Problem is still here

Can you provide a sample app that reproduces the error then so we can take a look? Thank you!

@josevalim Jose, here it is: https://github.com/asiniy/ecto_state_machine/pull/1
btw, how can I offer article for elixir-radar?

You are right. This feature won't work. "priv" should always point to a subdirectory inside priv. I will update the docs to make it clear. Thank you for trying it out and uncovering the limitation. :)
About elixir-radar, you can send an e-mail to elixir dot radar at plataformatec dot com dot br.
Finally, I would recommend you to revisit the state machine for Ecto. Ecto 2.0 is moving away from the idea of models and callbacks to focus on data. You are even invoking repos from inside the schema, which is a big no-no even in Ecto 1.0 times. :)

Thanks, it works!
@josevalim I can't actually understand intention of this limitation. Why can't I place under another directory, than priv?
How can I implement state machine via Ecto 2 in your opinion?

@asiniy we need to rely on the information being inside the _build directory so it works on umbrella directories and so on. And only priv and ebin are copied to _build.
I think a state machine should be implemented for Elixir data, it is not related to Ecto 2.0 at all.

@josevalim interesting idea, I'll move state machine from ecto namespace just to maps.

Howdy. Love Elixir. Lots of love in the documentation, which is the best part IMO. In that vein, I wanted to point out that the docs for Ecto.Migrate and Ecto.Gen.Migration still don't mention the requirement that the folder must be a subdirectory in the priv folder. I've forked and committed a possible tweak to the documentation. Should I go ahead and create a pull request?
Also, if the directory is invalid, I would think the mix ecto.gen.migration should fail.

Yes, please send a pull request! Docs improvements are always welcome!


allow auth option

Based on my reading of url_parser.js looks like there's no reason why auth isn't a supported option. Having support for this option would be very helpful for issues like https://github.com/Automattic/mongoose/issues/5419, otherwise we'd have to mutate the user-provided URI or ask users to not use those options anymore.

@vkarpov15 looks like this breaks all the tests! In my recent commit I did support merging user-provided options with the dbOptions, but not generically with the parsed object - this might be why this is failing?

Funny, the tests did pass locally for me. I'll debug in a bit but I'm curious about your recent commit. Is there already a way to pass username and password with options rather than through URI? That's all I want to get out of this PR, there's just no documentation on how to do so.

Yeah, that commit explicitly resolved this ticket: https://jira.mongodb.org/browse/NODE-1052. We were documenting that we supported readPreference, but not actually doing so.  To that end, I'm fine with supporting user provided overrides for options supported in the URI, I just want to be very careful about (as I'm sure you know this is error prone).

I took a closer look and haven't been able to figure out why my tests succeed locally but fail on travis. Any hints on how I might be able to repro test failures locally or fix the issue that's causing the tests to fail?

@vkarpov15 truly bizarre. I have the same experience on my mac, as well as a number of linux VMs.. I put together a variant of your test, just copying and pasting the whole test with a few replacements and it worked that way: https://travis-ci.org/mongodb/node-mongodb-native/builds/252444817
Sorry for the trouble, but we're in the process of heavily grooming the tests for the driver right now. One of our interns is writing a mocha compat layer right now, so we'll be switching away from integra and hopefully catching a lot of small issues in the process.

@vkarpov15 also, I modified the actual test as follows:
      var opts = { auth: { user: user, password: password } };
      connect(configuration.url('baduser', 'badpassword'), opts, connectionTester(test, 'testConnectGoodAuthWithOptions', function(db) {
        db.close();
        test.done();
      }));

The original way you were testing was with configuration.url(user, password) overridden with the same values - not sure there was much value to testing that

@vkarpov15 would you mind refactoring the tests to reflect the changes I suggested? We can merge this as soon as it passes CI

@mbroadst so I made those changes and a few more, but still have not been able to figure out why auth fails on travis but not locally. I added a print statement and it looks like the _finalOptions that MongoClient passes down are identical, even on travis, but auth still fails for the latter. Do you have any other suggestions?
Also, I'm gonna be in NYC later this week, we should grab a coffee. Shoot me an email<EMAIL_ADDRESS>

@vkarpov15 sorry for the trouble, I think this is related to some problems with integra. The change I was referring to is generally encapsulated in this commit. Specifically, I think what you haven't tried yet is fully splitting the tests into two separate test cases, rather than using asOption().  I'm hoping our switch over to mocha will remedy these types of problems, thanks for the patience in the meantime.

Thanks for the suggestion, looks like the tests passed :point_up: what a weird issue :face_with_head_bandage:

@vkarpov15 hooray 🎉

:tada: any ETA on the next release?

@vkarpov15 I'll be releasing 2.2.31 by EOW, just want to provide a little more time for some potential fixes.


feat: ignore seeder sql file

Could you provide a link to the tool that uses these files?

Insert seed data using sql-migrate

@dongyu looking over the docs for sql-migrate, I can't find any reference to .seed.sql or .seeder.sql files. Do you know where support for these files lives?

I'm going to close this for the time being. Please feel free to respond with a link to the sql-migrate documentation that mentions seed SQL files.


Edit landing page content, inconsistent capitalization

Closing, customer should be able to edit content on their own and create pull requests.


[zd3d12] added helpers for CreateShaderResourceView, CreateUnorderedAccessView, SetGraphicsRootDescriptorTable and SetComputeRootDescriptorTable

This code is required (a bit misleading though). We are creating SRVs using bindless approach at the beginning of the heap. So, the resources are then accessed in the shader like this:
    Texture2D srv_ao = ResourceDescriptorHeap[0];
    Texture2D srv_base_color = ResourceDescriptorHeap[1];
    Texture2D srv_metallic_roughness = ResourceDescriptorHeap[2];
    Texture2D srv_normal = ResourceDescriptorHeap[3];

OK. Let me fix that

Fixed but bindless is indeed very misleading. It seems only useful if generating HLSL as having a human keeping the HLSL in sync with the code usage of heap is error prone.

Fixed but bindless is indeed very misleading. It seems only useful if generating HLSL as having a human keeping the HLSL in sync with the code usage of heap is error prone.

In general bindless is very powerful and provides minimum CPU overhead. I was just lazy in this sample code and I assumed that we have only 4 SRVs and the beginning of the heap.
You can take a look at my new project which uses bindless in much better way (simple, fast and robust):
Header file which is used by both Zig and HLSL code and defines resource indices:
https://github.com/michal-z/zig-d3d12-starter/blob/main/src/cpu_gpu_shared.h#L1
In HLSL, resources are used like this:
https://github.com/michal-z/zig-d3d12-starter/blob/main/src/main.hlsl#L40
In Zig, SRVs are defined like this:
https://github.com/michal-z/zig-d3d12-starter/blob/main/src/gen_mesh.zig#L607


how to train five  lane lines

hi,Thank you for sharing，I want to know how to train five lane lines?

just modify dataloader and add one channel to featuremap


shared library support

Add capability to build .so lib.

@solomonik
Extending critter's support to pyCTF, I still think that critter needs to expose two functions at the python level: start(size_t) and stop(size_t,size_t). Otherwise, even if CTF can call start/stop inside its world creation and destruction, the data will add up across the entire program. As an example, there would be no way to use critter to accurately profile two different kernels in the same program if the start/stop only happen when the world object is created and destroyed.
If one simply wanted to profile everything in the entire file, then a pythonized critter::start/stop wouldn't be necessary of course.


Nuxt config sourcemap option ignored

Environment

Operating System: Darwin
Node Version:     v16.15.0
Nuxt Version:     2.16.0-27720022.54e852f
Nitro Version:    0.6.1
Package Manager<EMAIL_ADDRESS>Builder:          webpack
User Config:      sourcemap, serverHandlers, devServerHandlers, typescript, bridge, buildModules
Runtime Modules:  -
Build Modules<EMAIL_ADDRESS>
Reproduction
https://github.com/IlyaSemenov/nuxt-bridge-sourcemap-ignored
Describe the bug
The sourcemap option in nuxt config is completely ignored, both as an object or as a boolean.
Example:
export default defineNuxtConfig({
  sourcemap: { server: false, client: true },
})

nuxi build still generates server source map and doesn't generate client source map.
Additional context
The option is advertised by<EMAIL_ADDRESS>which is a direct dependency of @nuxt/bridge:
  /**
   * Whether to generate sourcemaps.
   *
   * @version 3
  */
  sourcemap: boolean | { server?: boolean, client?: boolean },

Logs
No response

(This option was introduced in Nuxt 3 and never added to Bridge.)


3script editor component

parent issue: https://github.com/threefoldtech/home/issues/1461
should be in a new package called threefold/ui

Created a simple form for the script editor using Tailwindcss. Will refactor the design tomorrow.

Edited the design so it would have line numbers and a copy button. currently working on converting the editor to a web component.

can't we use codemirror directly? https://codemirror.net

@xmonader We could but using Tailwindcss was one of the requirements. That's why I've built the editor from scratch.

Converted the editor to be a custom element which will be a web component once built. Also, included the signer and used tf-wallet-connecter to get user mnemonics. The component is ready to be used.

Added docs.

Edited script editor to use TF connect provider.

Verified,
Docs were added https://github.com/threefoldtech/tfgrid-sdk-ts/blob/development/packages/UI/README.md,
Script editor is working with mentioned steps in the docs, With threefoldSignerProvider and threefoldConnectorProvider.


Roadmap for hackage release?

I would just like to ask what the authors thoughts are for what remains to be done before this can be on hackage. Can you highlight any of the existing issues that need to be addressed? Are there some which are not listed? I'd like to start contributing in whatever direction will lead to this if I can.

@k0001

@martyall even though the current tisch code, as it stands, is expected to work correctly, I am not satisfied about some things and want to change them before moving forward with pushing this to Hackage (which comes with a commitment to maintenance).
The main two things I want to change are:

Use https://hackage.haskell.org/package/flay instead of HLists everywhere. The rationale for this is simplifying the API and UX, as well as improving error messages and field time access for large rows.

Do not rely on opaleye anymore. The rationale for this is avoiding the additional level of indirection which complicates the internals of tisch, as well as decoupling tisch roadmap from opaleye's roadmap.

After those those things are in place, I look forward to pushing a stable version to Hackage to be maintained for the foreseeable future.

Also, something else: I would like tisch to make use of linear types for its connection handling machinery http://www.tweag.io/posts/2017-03-13-linear-types.html

Thanks for getting back. I can't tell if any of the current branches are working toward these goals, but again if there is some kind of roadmap for working on these stories I'm glad to try to help if you want it. All I would need is maybe some kind of story board, or list of checkboxes, or whatever. Keep me posted.

@martyall no, I haven't started any work towards this yet. I'll try and make the process public so that you and others can get involved. Thanks :)

I'm using tisch at our company knowing the API will be in flux for awhile. We'd be happy to help test and submit any bug reports as things change.


Add Gulp JS Minification Task and Developer README

Added a Gulp task to do the JavaScript minification and a developer README to help with getting the site up and running locally.
I would like to find a way to run the jekyll serve and jekyll watch commands via gulp (so that everything gets done in one command), but I can create a separate issue for that.

This fixes #29

Sounds good to me!


Support Pluggable Configuration Generators

Just like we can support multiple service watchers, now we can support
multiple configuration generators which can output in whatever format we
want

@igor47 This PR is the start of a refactor that I hope will make it so that Synapse can take registrations from any registry and make them appear in whichever load balancing solution customers want.
Why?
HAProxy has some drawbacks for simple applications that don't need super powerful load balancing features and the community is definitely moving to polyglot load balancing solutions. In particular, nginx is starting to look much more viable as the open source version supports more and more load balancing features (and hitless reloads out of the box). Also it's always been super gross IMO that service watchers are coupled through to haproxy, so I'm trying to clean that up a bit.
What's Done?
Just like watchers folks can dynamically add ConfigGenerators which respond to config_updates calls from synapse. I have cleaned up the interface a bit.
What's TODO?
I have to figure out how to decouple haproxy from the ServiceWatchers. I'm thinking of exposing the generators through a function on the @synapse object that ServiceWatchers get at construction, and maybe do the validation there. The thing I'm primarily struggling with is that things like the Docker watcher reach into the haproxy implementation to figure out which port they need. I'm working on decoupling that as well.

@igor47 so this is one strategy that I think will work. I'm working on cleaning up the specs and code, but the current diff shows my current idea (using the synapse object to communicate between the two abstractions, but only in a limited fashion).

@igor47, @jnb  Alright, I think that this is in a fairly reasonable state for review :-D
Manual tests and specs give me reasonable confidence this works, but I can always add more specific testing if you like.

ugh sorry i've been so slow on reviewing this. this PR is very relevant to my interests, but it's been kind of a crazy week. i'm about half-done and i'll finish going through it tomorrow

No problem, I've been mostly hacking on this for fun so there isn't too much time pressure (the nerve change actually is hurting us right now heh). Mostly doing this because I want to allow services to be "discovery only" like cassandra which we will never talk through HAProxy for but we are still managing it in HAProxy because Synapse requires it to be managed in HAProxy heh.
Separately some folks at work are huge nginx fanboys ("it reloads without dropping traffic", "it can do retries of requests", "it can scale to more cores", etc ...) and I wanted to see what the reality was. Well after writing this proof of concept it does seem like it could work, but we'd have to reload for every up/down/flapped healthcheck (since it doesn't have the equivalent of a stats socket unless you pay for nginx+). I think that I might be comfortable putting marathon based HTTP services through it though because those already require restarts on every state change (marathon is a jerk about keeping ports the same) and they typically want non spec compliant things like retry-ing HTTP requests.

@igor47 now that envoy seems to be out and about (and it looks pretty sweet), what are your thoughts on this PR?

@igor47 @jnb gently poke
I think this branch is good for merge and I'm about 80% of the way through a nginx adding branch that will give us an e2e test of this functionality (synapse managing multiple proxies simultaneously).
@vulpine @bobtfish feel free to review if you want nginx fronted haproxy goodness :-D

@igor47 I took another pass, let me know if there are any remaining things you want me to fix in this iteration :-)

@igor47 I agree more refactoring is possible (especially in the specs overall), but I'd like to get this into master and start getting production experience with it asap since it's just a major refactor (and likely to have subtle bugs).
Are you ok to merge and then I can follow up with any bugs I find (and specs to match)?

ok, merging. i'll try to get this into some prod roles at airbnb this week, and i'll also take a look at the specs

Sweet thanks, I'll start rolling this out here ASAP and making sure I didn't break anything :-/


Adding in App Setup Policy docs

OPS Build status updates of commit f0786d9:
:clock10: Preparing: average preparing time is 1 min(s) 12 sec(s)

OPS Build status updates of commit f0786d9:
:clock10: Incremental building: average incremental building time is 12 min(s) 23 sec(s)

OPS Build status updates of commit f0786d9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Grant-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsPinnedApp.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsCallParkPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsPinnedApp.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsPinnedApp.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

OPS Build status updates of commit fc3fe33:
:clock10: Preparing: average preparing time is 1 min(s) 12 sec(s)

OPS Build status updates of commit fc3fe33:
:clock10: Incremental building: average incremental building time is 12 min(s) 23 sec(s)

OPS Build status updates of commit fc3fe33:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Grant-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsPinnedApp.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsCallParkPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsPinnedApp.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsPinnedApp.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

OPS Build status updates of commit 3b60ee8:
:clock10: Preparing: average preparing time is 1 min(s) 12 sec(s)

OPS Build status updates of commit 3b60ee8:
:clock10: Incremental building: average incremental building time is 12 min(s) 24 sec(s)

OPS Build status updates of commit 3b60ee8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Grant-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsPinnedApp.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsCallParkPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsPinnedApp.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsAppSetupPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsPinnedApp.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


changes made for cancel query button

You're moving in the right direction, but I think that just CompletableFuture is not enough. You may need to keep track of Threads in which queries are executed and interrupt them directly.


[Bug] JavaScript errors using the UMD Package Generation Tool code

Before you submit this issue, have you checked the following

[X] Is this really a problem?
[X] I have searched the Github Issues for similar issues, but did not find anything.

Affected packages and versions
@univerjs
Reproduction link
I tried to use the UMD Package Generation Tool to create a minimal demo. Even though the demo loaded, it produced errors in the console. Steps to reproduce:

Go to https://univer.ai/guides/quick-start/umd/ and download the generated univer.umd.css and univer.umd.js files.
Save the boilerplate code to index.html.
Add the CDN links as well (copy code exactly as shown). The final index.html is attached: index.zip
Load this index.html.

Expected behavior
A basic spreadsheet component with no JavaScript errors.
Actual behavior
These errors showed up in the console:

System information
System:
OS: Windows 10 10.0.19045
CPU: (8) x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
Memory: 14.94 GB / 31.95 GB
Binaries:
Node: 18.18.0 - C:\Program Files\nodejs\node.EXE
npm: 9.8.1 - C:\Program Files\nodejs\npm.CMD
Browsers:
Edge: Chromium (123.0.2420.97)
Internet Explorer: 11.0.19041.3636

It seems that these two packages are missing.
+ <script crossorigin="anonymous"<EMAIL_ADDRESS><script crossorigin="anonymous"<EMAIL_ADDRESS>+ <script crossorigin="anonymous"<EMAIL_ADDRESS>
https://stackblitz.com/~/github.com/awesome-univer/sheets-umd-demo


Add note to devguide re: Snap incompatibility

Plugin doesn't work w/ Snap installation, add note.
See note here.

needs a rebase, my fork is missing the right branches. :facepalm:


Fix variation-module-options parsing in numeric.py

removes unnecessary 'for' loop during numeric module options processing
the aforementioned loop led to an exception before this fix: 'ValueError: too many values to unpack (expected 2)' Exception was thrown since the utils.split() method returns a flat  list and not a dict[list] or the like

 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.


Build and train machine learning models for tasks like image recognition, sentiment analysis, or recommendation systems

Hey, I would like to work on this issue. Can I get assigned to this?

Before we proceed, we kindly request you to:

Star this Repository: ⭐ Starring the repository helps us reach a wider audience and encourages more contributions. You can star the repo by clicking the "Star" button at the top of this page.

Follow the Code of Conduct: 📜 We maintain a Code a Code of Conduct to ensure a positive and respectful environment for all contributors. Please review and abide by it throughout your involvement in this project.

Add Your Profile Details: 📋 You can find instructions on how to add your profile details to the repository in our YouTube Video. Please follow the steps outlined in the video to complete this task before contributing.

If you have any questions or need further assistance, feel free to ask. Happy coding! 🚀


Deprecate transforms.RandomSizedCrop in torchvision example

Fixes #{issue number}
Changes

@NivekT Just want to confirm the mypy Error is caused by the change in PyTorch Core and not related to the PR.


Update cargo-flash readme

It would be great if someone could try the nRF52832, nRF52840 and STM32F042 flash targets :)

Can you rebase that please? Your PRs tend to include everything and the kitchen sink of previous changes.

I am aware. The problem is that I start a new PR before the old one is approved to keep em small.
But I still want the old changes without much hassle. Depending on the extent of the changes that will be merged to master it can be a hassle to merge after, so I make a new branch off the branch of the most recent PR. And before I submit the PR to final review I rebase :)

The problem with that approach is that noise makes reviewing hard if not impossible.

@therealprof I am aware :) that's why i didn't request a review yet :) should have marked it draft maybe then.


add new link of the origin of SSI

Signed-off-by: Kyle Den Hartog<EMAIL_ADDRESS>

Do you know which commit is blocking the merge? From what I can tell all of my commits are DCO signed.

9ab3b5e4dd9d49 is signed off but not signed

Ok I finally got around to adding GPG signing. This PR Should be good now.

Does anyone know why a 1 line change is causing the codecov to drop so significantly? I've not worked with this tool before, so I'm trying to figure out what the cause of it is. I suspect it has to do with the additional code added since I got the signing fixed. I may just close this PR and reopen a new one since it seems the easiest fix.

@kdenhartog  codecov frequently gets confused about that, clearly it's not correct here. Feel free to open a new PR instead.


Running seed:certified-user should add the user's Microsoft account to the DB

Describe the Issue
When running the dev environment as a certified user, we have all C# trophies completed but the user's MS transcription link is not linked:

While this is a possible scenario (the user links their account to claim trophies, then unlinks it), it seems like an edge case to me, and it shouldn't be the default behavior when we first setup the dev environment.
Affected Page
Any C# trophy page. Example: /learn/foundational-c-sharp-with-microsoft/write-your-first-code-using-c-sharp/trophy-write-your-first-code-using-c-sharp
Steps to Reproduce

Seed the local environment with seed:certified-user
Go to /learn/foundational-c-sharp-with-microsoft/write-your-first-code-using-c-sharp/trophy-write-your-first-code-using-c-sharp
See the "link MS account" section on the page

Expected behavior
The seed:certified-user command should add the user's Microsoft account / transcription link to the DB.
And the trophy pages should look similar to this:

Additional context
This issue was discovered here: https://github.com/freeCodeCamp/freeCodeCamp/pull/52445#discussion_r1420122664.

what link should be added?

Thanks, @huyenltnguyen, I agree that this is a weird default. The fix should just be to add an MsUsername record with a userId of the certified user. @moT01 can you confirm/deny?

Confirmed.
Here's what my entry looks like when I link my account locally:
  {
    _id: ObjectId('65785b25d4c5bd0565c0184d'),
    userId: ObjectId('5bd30e0f1caf6ac3ddddddb5'),
    ttl:<PHONE_NUMBER>0,
    msUsername: 'moT01'
  }

We don't save the transcript link anywhere, so I'm fine with using my username if we want.


Add Rust to SCons

Add Rust to the SCons build system. If a Cargo.toml file is present then it builds with Cargo, otherwise it will build with rustc. As such a Cargo.toml file is necessary if any external crates are necessary (usally an extern crate ...; in the source file). This could be replaced with a custom Scanner, but that's probably overkill and error prone.
In order to have all the code files get picked up by SCons, I have the *.rs and the Cargo.toml file in the same directory, and define the path to the source file in Cargo.toml. An example of this is the Cooley-Tukey code.
I have the *.pdb files and the target/ directories marked by SCons for cleaning when using the -c argument. This deletes the Cargo.lock files, causing cargo to recompile the external crates on the next build.
Due to Cargo putting the executable into a directory in the target directory, it is built in the same place as the source file and the executable then copied to the build/ directory. There is a command line argument to copy the build artifacts to a directory (--out-dir) but this is only available on nightly rust. When (and if) this gets stabilized then it would simplify the SConstruct file to take advantage of this.

Cooley-Tukey and Split Operator Method don't compile, but this is a problem with those files rather than SCons. I've already
opened an issue for it.

Dockerfile install looks fine to me.

I've fixed the issues with some programs not compiling due to the wrong library versions being used. Solves #944 . It could do with updating, but that's another issue.


setState({value, 'a long long text') in ios multiline textInput won't wrap in multiline

If the text is too long for the textInput line, it won't be wrapped in multiline automaticly until you edit something.
For example setState({value, 'a long long long long long long long long long long text'), you will only see a long long long long ... if the lineView is not that long, not a muliline wrapping text.
This issue doesn't happen when set the initial state.
And it only happens on ios.
I check the code in TextInput.js. The android  <AndroidTextInput> support multiline but ios use two different view as <RCTTextField> and <RCTTextView>. It uses <RCTTextView> to implement multiline.
Maybe setting the text value on it directly doesn't  trigger the wrapping action. But I could not figure it out.

I finally figure out what's wrong.
In RCTTextView.m
- (void)setText:(NSString *)text
{
  NSInteger eventLag = _nativeEventCount - _mostRecentEventCount;
  if (eventLag == 0 && ![text isEqualToString:_textView.text]) {
    UITextRange *selection = _textView.selectedTextRange;
    _textView.text = text;
    [self _setPlaceholderVisibility];
    _textView.selectedTextRange = selection; // maintain cursor position/selection - this is robust to out of bounds
    [self updateContentSize]; //  ---------------MISSING THIS-------------
  } else if (eventLag > RCTTextUpdateLagWarningThreshold) {
    RCTLogWarn(@"Native TextInput(%@) is %zd events ahead of JS - try to make your JS faster.", self.text, eventLag);
  }
}

@narychen Up for sending a PR?

@satya164 Just one line code, maybe the one who write this file could check for it.

@narychen A PR is welcome though :)

PR has been made.  https://github.com/facebook/react-native/pull/5247

this bug happened again on version4.2.
this time setText function did not operated, instead doing - (void)setTextStorage:(NSTextStorage *)textStorageon RCTText.m
so, how to fix it?


mobile template

Hello all,
I want to use the mobile template. i can select the mobile template. however when i try to send a nzb to sabnzbd it is not working. The loading icon keeps circling arround and arround.
Does anyone have a sollution for this?
Thanx in advance.

same as https://github.com/Spotweb/Spotweb/issues/37


Remove AppsAndFeaturesEntries from HydrusNetwork.HydrusNetwork v506

Microsoft Reviewers: Open in CodeFlow

URL:
Invalid URI: The hostname could not be parsed.
(Automated message - build 891)


This library does not return status code to protect parsing error

Hi Lucas,
I was trying to download and parse an article from a site with response status code other than 200(OK)
When parsing I receive error message. But I am not able to protect my script by handling exception or protecting depending on status code.
Below is my sample script where sample url returns 416 response. And I have no ways to protect script
url = 'https://www.chronicle.com/'
article = Article(url)
article.download()
##TODO: if else or exception handling to protect script
article.parse()

We can handled through ArticleException


openidc_discover(): issuer field in Discovery data does not match URL,

@Trojan295
We are getting below issue. we installed plugin successfully and then configured one api with oidc plugin. After than when we hit from kong proxy to that api we are getting below issue.
LOG  2018/11/21 16:14:22 [error] 15837#0: *35 [lua] openidc.lua:492: openidc_discover(): issuer field in Discovery data does not match URL, client: <IP_ADDRESS>, server: kong, request: "POST /imapi HTTP/1.1", host: "<IP_ADDRESS>:8118"
Config we did.
"scope": "openid",
"client_id": "7db76fea-f48d-4396-89da-632625c6a435",
"discovery": "https://xx.xx.xx.xx/adfs/.well-known/openid-configuration",
"client_secret": "gahX1Oe_LNnaMb0dntiMLdQQ_8kPGLkOaakT2Npj"

},

Is the issuer field in the discovery document correct? It should match the URL on, which the document is available.

@Trojan295
Can you please help to get this close.
Our configuration in kong plugin is like below.
**    "client_id": "7db76fea-f48d-4396-89da-632625c6a435",
"bearer_only": "no",
"ssl_verify": "no",
"discovery": "https://TEST(uppercase).pocad.com/adfs/.well-known/openid-configuration",
"client_secret": "gahX1Oe_LNnaMb0dntiMLdQQ_8kPGLkOaakT2Npj"**
when we hit (https://test.pocad.com/adfs/.well-known/openid-configuration) URL from browser we are getting below response.
Response
{"issuer":"https://TEST(Uppder case).pocad.com/adfs","authorization_endpoint":"https://test.pocad.com/adfs/oauth2/authorize/","token_endpoint":"https://test.pocad.com/adfs/oauth2/token/","jwks_uri":"https://test.pocad.com/adfs/discovery/keys","token_endpoint_auth_methods_supported":["client_secret_post","client_secret_basic","private_key_jwt","windows_client_authentication"],"response_types_supported":["code","id_token","code id_token","id_token token","code token","code id_token token"],"response_modes_supported":["query","fragment","form_post"],"grant_types_supported":["authorization_code","refresh_token","client_credentials","urn:ietf:params:oauth:grant-type:jwt-bearer","implicit","password","srv_challenge"],"subject_types_supported":["pairwise"],"scopes_supported":["email","user_impersonation","openid","aza","winhello_cert","profile","allatclaims","logon_cert","vpn_cert"],"id_token_signing_alg_values_supported":["RS256"],"token_endpoint_auth_signing_alg_values_supported":["RS256"],"access_token_issuer":"http://test.pocad.com/adfs/services/trust","claims_supported":["aud","iss","iat","exp","auth_time","nonce","at_hash","c_hash","sub","upn","unique_name","pwd_url","pwd_exp","mfa_auth_time","sid"],"microsoft_multi_refresh_token":true,"userinfo_endpoint":"https://test.pocad.com/adfs/userinfo","capabilities":[],"end_session_endpoint":"https://test.pocad.com/adfs/oauth2/logout","as_access_token_token_binding_supported":true,"as_refresh_token_token_binding_supported":true,"resource_access_token_token_binding_supported":true,"op_id_token_token_binding_supported":true,"rp_id_token_token_binding_supported":true,"frontchannel_logout_supported":true,"frontchannel_logout_session_supported":true}.
Questtion 1: Is issuer URL and discovery URL is case sensitive and should match URL String?
Question 2:  We change discovery URL in kong plugin same with Issuer URL( i mentioned URL of Issuer as TEST.pocad.com/adfs. and configured discovery URL in kong plugin https://TEST.pocad.com/adfs/.well-known/openid-configuration) and hit to kong proxy we are getting some Login page in response.

Error

JavaScript required
JavaScript is required. This web browser does not support JavaScript or JavaScript in this web browser is not enabled.
To find out if your web browser supports JavaScript or to enable JavaScript, see web browser help.

POCADFS Login Page

An error occurred
An error occurred. Contact your administrator for more information.

Error details

Activity ID: 78f6d3f9-2c40-4e21-0600-0080000000fa
Error time: Thu, 22 Nov 2018 06:48:59 GMT

Set-AdfsWebTheme -TargetName  -AdditionalFileResource @{uri='/adfs/portal/images/hrd.jpg';path='.\hrd.jpg'} // //if (typeof HRD != 'undefined') { // SetIllustrationImage('/adfs/portal/images/hrd.jpg'); //} //]]>

But in KONG log we are getting (openidc_discover(): issuer field in Discovery data does not match URL, client: <IP_ADDRESS>, server: kong, request: "POST /imapi HTTP/1.1").
Thanks in advance!!

The issuer field must be a substring of the discovery URL you put in the plugin configuration. It is case sensitive. Could you try to provide more log by running Kong with debug level logs? You can also try to put the whole Discovery URL in lowercase characters.
Besides, updating lua-resty-oidc to 1.9.0 in this plugin would solve also the problem, cause they remvoed checking this field. https://github.com/zmartzone/lua-resty-openidc/issues/219

@Trojan295

We have configured below details in kong-oidc plugin. And now issuer URL issue got resolved. We are now getting some "session_secret" not found for writing; maybe it is a built-in variable that is not changeable or you forgot to use "set $session_secret '';" in the config file to define it first. We gone through https://github.com/nokia/kong-oidc/issues/1 and set session_secret (encoded password) in plugin but still we are facing below issue.  Can you please help to get this close.
Config we did for plugin
curl -i -X POST --url http://localhost:8115/apis/im/plugins
--data 'name=oidc'
--data "config.client_id=30f547df-2bdd-4fc9-a4e7-7c21cadf6ec8"
--data "config.client_secret=e8uYsjPbljp238tyJeHNWh72t33osS8jCQ6xyRUp"
--data "config.discovery=https://test.pocad.com/adfs/.well-known/openid-configuration"
--data "config.session_secret=dGliY29AMTIz"
** DEBUG LOGS**
"session_secret" not found for writing; maybe it is a built-in variable that is not changeable or you forgot to use "set $session_secret '';" in the config file to define it first
stack traceback:
coroutine 0:
[C]: in function 'error'
/usr/local/openresty/lualib/resty/core/var.lua:114: in function '__newindex'
/usr/local/share/lua/5.1/kong/plugins/oidc/session.lua:11: in function 'configure'
/usr/local/share/lua/5.1/kong/plugins/oidc/handler.lua:19: in function </usr/local/share/lua/5.1/kong/plugins/oidc/handler.lua:14>
coroutine 1:
[C]: in function 'resume'
coroutine.wrap:21: in function coroutine.wrap:21
/usr/local/share/lua/5.1/kong/init.lua:468: in function 'access'
access_by_lua(nginx-kong.conf:97):2: in function <access_by_lua(nginx-kong.conf:97):1>, client: <IP_ADDRESS>, server: kong, request: "POST /imapi HTTP/1.1", host: "xx.xx.xx.xx:8118"

Then we tried to set set $session_secret '';" in /usr/local/kong/nginx-kong.conf file and restarted kong. But after restarting  kong same property is getting deleted automatically. Hence we are getting same issue.
"session_secret" not found for writing; maybe it is a built-in variable that is not changeable or you forgot to use "set $session_secret '';" in the config file to define it first
stack traceback:
coroutine 0:
[C]: in function 'error'
/usr/local/openresty/lualib/resty/core/var.lua:114: in function '__newindex'
/usr/local/share/lua/5.1/kong/plugins/oidc/session.lua:11: in function 'configure'
/usr/local/share/lua/5.1/kong/plugins/oidc/handler.lua:19: in function </usr/local/share/lua/5.1/kong/plugins/oidc/handler.lua:14>
coroutine 1:
[C]: in function 'resume'
coroutine.wrap:21: in function coroutine.wrap:21
/usr/local/share/lua/5.1/kong/init.lua:468: in function 'access'
access_by_lua(nginx-kong.conf:97):2: in function <access_by_lua(nginx-kong.conf:97):1>, client: <IP_ADDRESS>, server: kong, request: "POST /imapi HTTP/1.1", host: "xx.xx.xx.xx:8118"
So we tried below scenario to set session_secret but all cases we received same error log  .
a. Provided session_secret  in plugin config.
b. tried to set as env variable export kong_session_secret=XX
c. Tried to set set $session_secret ''; in /usr/local/kong/nginx-kong.conf file
d. tried to set session_secret in /etc/kong/kong.conf file.

Can you provide details about:

which Kong version are you using
which plugin version are you using
are you running Kong in HA or single node?

@Trojan295
Pfb details for your reference.

We are using kong 0.14.1 CE
We installed kong-oidc 1.1.0 as we use "luarocks install kong-oidc " command to install kong-oidc plugin as it was mention in installation step.
For development we are using single node, but in production we will be using HA Kong .

Regards

The logs look good now. The header needs to be: "Authorization: Bearer <>".
I haven't tested this plugin against ADSF 3.0.

@trojan295
we are not able to get response from upstream url after send Authorization: Bearer <>". But if we diable oidc plugin upstream url works fine. No error getting printed in error log.
Regards

You would need to set the config.introspection_endpoint, if you want to enable passing the token in the Authorization header directly.

@Trojan295
We are not trying to pass Authorization header directly to upstream api.
we are going As per your design diagram. We are not able to get the point where actually we stopped to reach to upstream Api. As per your previous comment kong and adfs connection is good.
Can you please help to get answer for below point?

Client application need to send authorization grant code to kong proxy ( which parameter client application need to send in header Authorization Code<<>>  to kong proxy? )
Then kong will exchange grant for Access token and ID token with adfs (how we can make sure it is happening as there is no log?).
After getting Access token , kong will try to get x-userinfo from adfs using access token( not sure it is happening or not? as there is no enough log)
After getting x-userinfo from adfs , kong-oidc plugin will invoke upstream api and send response back to client.

Now client application is sending Authorization Code<<>> in header to kong proxy but they are getting below response where as upstream API is working fine.
**HTTP/1.1 500 Internal Server Error
Date: Mon, 17 Dec 2018 07:28:03 GMT
Content-Type: text/plain; charset=UTF-8
Transfer-Encoding: chunked
Connection: close
Server: kong/0.14.1
An unexpected error occurred**


Export to sanitized (Pandoc-ready) markdown file

I recently was able to run Pandoc on to generate PDF right off the markdown notes in my vault.
With a bit of modification to the .md note, one can—using pandoc-xnos filter—add figure/table/equation/section numbers (and image captions). This is hugely helpful in cross-referencing within the text, and there would be no need to add a figure environment (#19). (There was an issue I had with equation environment when aligned environment was used in math blocks, and it screwed the rendered math in the compiled PDF.)
Moreover, footnotes are supported, too, and Pandoc takes care of citations (haven't tried yet) and makes quotes with curly quotation marks.
It would be great if the plugin had a command to export to a sanitized markdown file, wherein embedded notes were "flattened" (the way the plugin currently pastes the content of the embedded note), links sanitized, and image (relative) paths were corrected—both of which are taken care of already.
Pandoc requires an additional change to the way images are embedded:
The embedded  image ![[figure.png]] should be converted to ![Alt text](/path/to/figure.png "Caption") or ![Caption](/path/to/figure.png). Adding {#fig:some-label} after the image would allow cross-referencing possible by calling @fig:some-label in the text.
Solutions I have in mind
Since Obsidian supports ^ for block referencing, (and given that  it now supports %%comments that are not rendered in the View mode%%), I suggest using the following syntaxes that are compatible with Obsidian's View mode, thus not screwing the rendered text of the original note:
Desired feature behavior for figures
I personally would love if I can embed an image as
%%My caption here%%![[figure.png]]^my-sad-label

And it is converted to the following in the exported .md file:
![My caption here](/path/to/figure.png){#fig:my-sad-label}

Desired feature behavior for equations
One can append {#eq:my-label} at the end of the math block in the original note, which breaks the rendered math in View mode
$$
\Gamma = \sum_{i=1}^N i^2
$${#eq:a-terrible-equation}

Alternatively, if the ^ referencing (with a line break after the closing $$) would help, without making any problem in View mode:
$$
\Gamma = \sum_{i=1}^N i^2
$$
^a-terrible-equation

Are these feasible features?

(I truly appreciate your efforts and quick fixes, and I wish I could contribute by forking and pull requests. Though I have no knowledge of TypeScript and js, unfortunately. ❤ )

Sorry for the delayed response. I've been very busy.
Yeah I originally looked at using psn doc but wanted a more integrated workflow. Nonetheless I think this is a good feature. I'm torn on whether this belongs in this plugin or whether it should be its own and the common logic should be refactored into a library. I'll try to take a look this weekend if I can.
In the meanwhile I recall there being a markdown sanitizer plugin but I don't remember what it was called.

Hi again and no worries—and sorry that I wasn't getting the notification for some reason.
You might be referring to the Markdown Prettifier plugin (right?), which only cleans the formatting (double spaces, coherent bullet points, etc.) What I meant by sanitization, however, was about "flattening" the embedded notes, cleaning links (such that [[ ... ]] do not appear in the output, i.e., changing [[note1.md|some note]] to some note), and correcting the path to embedded images.
I see your point on Pandoc, and that is a fair point – I can imagine Pandoc would perhaps disrupt the workflow. Though, making a Pandoc-ready markdown output shouldn't cause disruptions I guess (no? 🤔)

Yeah. I think there's a definite use for this feature. I think it would make sense to separate into two plugins sharing a common background. I'm unfortunately very busy right now. But I'll try to get started on it when I can.


Added landing page UI

@CodiumAI-Agent /review

PR Review

 ⏱ Estimated effort to review [1-5]
3, because the PR includes multiple file changes across CSS, JavaScript, and package configuration files. The complexity lies in understanding the UI changes and ensuring compatibility and functionality with existing code.

 🧪 Relevant tests
No

 🔍 Possible issues

Hardcoded Styles: The use of hardcoded pixel values and colors in CSS may lead to issues with responsiveness and theming.

Dependency Version Compatibility: The newly added dependencies should be checked for compatibility with the existing project setup and other dependencies.
 🔒 Security concerns
No

 Code feedback:
relevant fileClient/Client/src/pages/Main.module.css
suggestion

Consider using CSS variables for colors and font sizes to improve maintainability and support theming. For example, --primary-color: rgb(84,121,247); and use it as color: var(--primary-color); [important]

relevant linecolor: rgb(84,121,247);
relevant fileClient/Client/src/pages/Main.jsx
suggestion

Extract inline styles into the CSS module for consistency and to avoid the performance overhead of inline styles. For example, move the span style to Main.module.css. [important]

relevant lineLet's start our learning and earning journey!!
relevant fileClient/Client/package.json
suggestion

Ensure that the newly added react-scroll-motion package version is compatible with the current project setup, especially with the version of React being used. [medium]

relevant line"react-scroll-motion": "^0.3.3"
relevant fileClient/Client/src/components/LandingPage.jsx
suggestion

Use React Fragment shorthand <>...</> for consistency and readability, as it's already used in other parts of the code. [medium]

relevant line<>

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review which includes several types of feedbacks, such as possible PR issues, security threats and relevant test in the PR. More feedbacks can be added by configuring the tool.
The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.

When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:

/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:

[pr_reviewer]
some_config1=...
some_config2=...

See the review usage page for a comprehensive guide on using this tool.

Persistent review updated to latest commit https://github.com/kalviumcommunity/S47_Janhavi_Chauhan_Capstone_IntellectCoin/commit/7fe083266cf8bc0d25d6e963d22c499287bacf3e