Update contract naming scheme and cleanup docs.

Update contract naming scheme, and cleanup docs.
This updates the naming scheme for DAO DAO contracts. All contracts
are given the prefix cw-dao. The core module is renamed to be
cw-dao-core. Proposal modules are named in the format
cw-dao-proposal-<proposal_type> and voting modules are named in the
format cw-dao-voting-<voting_type>.
This also improves the README situation for many contracts.
Possibly controversially, this removes the cw-named-groups and
cw-names-registry contracts. The thinking being: our design
direction has changed since those were written. For the names
registry, I'm currently imagining that we will build out a much more
sophisticated system here when we do vote delegation that will replace
this contract. For named groups, we now support strings in
cw-dao-core items. This probably replaces the need for this contract
as frontends will likely just want to store this information in that
string.
Notably as well, this PR bumps our cargo requirement to the nightly
channel so that we may use the accepted-but-not-yet-mainline cargo
workspace dependencies
changes. This
has less-than-ideal UX as we will have to describe to contributors
that they must either rustup default nightly or run cargo +nightly
to interact with the contracts. It comes with what I see as
significant advantages. In particular:

We can unify package versions across packages and contracts
allowing us to enforce that we are using the same version of
CosmWasm dependencies everywhere.
We can unify package metadata across contracts which makes
maintaining shared metadata across packages simple.

I belive that (1) provides both ergonomic and security advantages as
we are unlikely to publish a package with outdated or insecure
dependencies.
It also allows us to enforce at the workspace level that packages are
using version = "*" when importing local contracts. Doing this is
important because it makes sure that we will detect breaking API
changes when updating one contract and keep our contracts working
together cohesively.
Overall, using workspace dependencies makes package versioning much
more cohesive at the expense of some additional cognative overhead
when first interacting with the contracts.
Remaining TODO

Make sure README and lib.rs package level documentation is in
sync for all crates.
Remove debug/cw-dao-proposal-sudo and
debug/cw-dao-voting-cw20-balance contracts as they are no longer
needed.
Investigate interaction between workspace-optimizer and cargo
nightly requirement. This may require a PR into workspace
optimizer.
Update typescript type generation code to handle nested contract
directories (contracts/proposal-modules and
contracts/voting-moudles).

The cargoworkspace-inheritance feature that we use here has been stabilized and will be released in the next Rust release: https://forge.rust-lang.org/#current-release-versions

Codecov Report

Merging #461 (c6414b1) into main (9df2d58) will decrease coverage by 1.01%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##             main     #461      +/-   ##
==========================================
- Coverage   95.81%   94.80%   -1.02%
==========================================
  Files          34       32       -2
  Lines        5448     5196     -252
==========================================
- Hits         5220     4926     -294
- Misses        228      270      +42

Impacted Files
Coverage Δ

contracts/cw-dao-core/src/contract.rs
93.37% <0.00%> (ø)

contracts/cw-dao-core/src/helpers.rs
100.00% <ø> (ø)

...osal-modules/cw-dao-proposal-multiple/src/state.rs
100.00% <ø> (ø)

...es/cw-dao-proposal-multiple/src/voting_strategy.rs
100.00% <ø> (ø)

...sal-modules/cw-dao-proposal-single/src/proposal.rs
97.90% <ø> (ø)

...ewards/cw20-stake-external-rewards/src/contract.rs
98.91% <ø> (ø)

...ards/cw20-stake-reward-distributor/src/contract.rs
99.14% <ø> (ø)

contracts/utils/cw-admin-factory/src/contract.rs
95.45% <ø> (ø)

contracts/utils/cw-token-swap/src/contract.rs
95.04% <ø> (ø)

contracts/utils/cw-token-swap/src/state.rs
100.00% <ø> (ø)

... and 33 more

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

Closing for now. I think we've landed on good names here.
I'm going to roll back using the cargo workspace-dependencies feature. Despite that feature being slated for stabilization in the next release, it is still in beta so using it would require that we use a beta version of the Rust compiler. I think we could make a compelling case for this being OK, but explaining that when we're deploying these contracts on permissioned chains may be politically sub-optimal.
Will re-open once we're closer to the v2 release again.


compile tf with bazel with/without jemalloc?

Hi, did  you compile tensorflow with or without jemalloc ?

No, I did not.


Possible to enhance/change where the popups come?

Thank you so much for this plugin, I've needed it for a long time!
The notifications work well, but they are quite discreet.. which causes me to frequently miss them. I have a large screen, and loads of popups re start up and such that have numbed me from that 'upper corner' flickering/notes.
Would it be possible to create more of an 'in your face'-notification? Either by placement or background color (I use dark mode), together with a direct link to the Inbox(es) that have been changed? Perhaps keep a flag on those Inboxes that haven't been processed to 'force'/remind me to take care of them at a later time?

I currently set the notification to not go away until I click on it, so that way when the noise clears I'm left with my inbox notification still in the corner.
If it's an inbox note, you can be taken to the inbox note by right clicking on the notification. You can't really link to a folder so I didn't add that functionality for inbox folders.
I like the idea of having a flag or something on the inbox note/folder in the file explorer, though I worry about plugins or themes conflicting with that, so I'm hesitant to go with that option.


Need help on costing for this solution for small , medium & enterprise customers.

Any insights on running costs for this solution?

Hi @jasonwbaxter the solution as is, doesn't have a direct cost, only the Logs Analytics consumption and Power BI if you want to create reports using that solution. The variables related to this solution are several, because MPARR permit to collect all the activities from end users, admin users and services running in the background, in that order of ideas, having E5 instead of E3 generates more logs, and that is more data ingested to Logs Analytics, other factors like if you are using several or few DLP rules, or labels, or MIP Scanner, Auto-Labeling,
But during this week I can share some statistics that can be helping to have an idea.

Hi @ProfKaz  , Thank you so much for the response and i 100% understand how varied it can be , anyway you can give a rough estimate for 1000 users on e5 and then the customer can at least set a budget on the Azure Resource group.

In the image you can see some estimate cost with the variables based on some customers statistics, I uploaded an Excel file that can be used as a calculator in this link https://github.com/microsoft/Microsft-Purview-Advanced-Rich-Reports-MPARR-Collector/blob/main/Support Information/MPARR - Cost calculator DRAFT.xlsx


Support MSA Auth

Currently AzureBoth only supports authentication using Azure Active Directory accounts, not Microsoft accounts (i.e. Outlook, Hotmail, etc).  We depend on the AuthBot https://github.com/matvelloso/AuthBot, so this functionality enhancement should happen there.  Enhancement would be making auth work with MSA.
Bugs would be not graceful handling of when you authenticate with MSA account to let people know we don't support MSA.

MSA is global admin of tenant - you get: "message": "An error has occurred.", "exceptionMessage": "AADSTS65001: The user or administrator has not consented to use the application with ID '88443132-9b52-485d-90ed-49f37efd1c2b'. Send an interactive authorization request for this user and resource.”
MSA is just a user in the directory - you get an infinite spinning wheel like this:

I have the same problem, and it doesn't work on some AAD accounts also, specifically ones replicated from on premises.

@aloneguid are the AAD accounts replicated from on-premises which don't work global admins by chance?

@dtzar no, they aren't definitely. It works randomly on some subscriptions

At the moment with the current code there's no workaround, you'd have to create a user in your AAD domain and use that to sign in.
One flow you can implement is to ask the user for their subscription ID, then use the subscription ID to determine what tenant the user is in, then finally, sign the user in to that tenant using the / endpoint instead of /common.
See:  https://azure.microsoft.com/en-us/documentation/articles/resource-manager-api-authentication/
This would work for @hotmail.com and other consumer Microsoft accounts, but would have to be a version of the bot that's not hosted by Microsoft.


Fix optional types in queries

Fix for #3638
Related #3463 #3060 #2590
This MR makes all nullable fields be optional (for typescript and flow), if avoidOptionals option is not true.
There is an inconsistency between the top-level types and the query types. Example from dev-test:
https://github.com/dotansimha/graphql-code-generator/blob/master/dev-test/githunt/types.preResolveTypes.ts
// This is the top-level type and description is optional
export type Repository = {
  __typename?: 'Repository';
  /** Just the name of the repository, e.g. GitHunt-API */
  name: Scalars['String'];
  /** The full name of the repository with the username, e.g. apollostack/GitHunt-API */
  full_name: Scalars['String'];
  /** The description of the repository */
  description?: Maybe<Scalars['String']>;
  /** The link to the repository on GitHub */
  html_url: Scalars['String'];
  /** The number of people who have starred this repository on GitHub */
  stargazers_count: Scalars['Int'];
  /** The number of open issues on this repository on GitHub */
  open_issues_count?: Maybe<Scalars['Int']>;
  /** The owner of this repository on GitHub, e.g. apollostack */
  owner?: Maybe<User>;
};

// This is query type, and description is mandatory now
export type CommentQuery = {
    repository: {
      __typename?: 'Repository';
      description: Maybe<string>;
      open_issues_count: Maybe<number>;
      stargazers_count: number;
      full_name: string;
      html_url: string;
    };
  }>;
};

Same bug for preresolveTypes: false
Example from https://github.com/dotansimha/graphql-code-generator/blob/master/dev-test/githunt/types.ts
// This is top-level type
export type Query = {
  /** Return the currently logged in user, or null if nobody is logged in */
  currentUser?: Maybe<User>;
};

// This is query type, notice currentUser become mandatory
export type CommentQuery = { __typename?: 'Query' } & {
  currentUser: Maybe<{ __typename?: 'User' } & Pick<User, 'login' | 'html_url'>>;
}

My fix is actually easy. There are two methods: formatNamedField and wrapTypeWithModifiers, they should be applied always together, and formatNamedField should add optional mark when it needed (line 53)

Hi @standy !
This seems like a big PR with more changes, can you please explain your changes there?
because the bug effected preResolveTypes types only as your said, but it seems like all examples under dev-test are updated with changes, and you changed some more files?
Thanks!

Hello, @dotansimha
I did update my PR. And I found a few similar issues, which is fixed here.

Just went through the changes in this PR. This is great!
Thank you so much @standy  :)

🚨 I think this "fix" was incorrect. It seems to break the behavior described here: https://github.com/dotansimha/graphql-code-generator/issues/3060#issuecomment-570174939
We tried upgrading to 0.13.1 and noticed a lot of client-side generated query types were now suddenly "optional". A field that is present in the client-side query should never be "optional", although its value may be null if it is not marked non-nullable.
So this PR introduced incorrect behavior and IMO should be reverted. Happy to discuss further - I know a lot of work went into it! It's a subtle distinction, but it's important not to confuse "client" and "server" types in implementations - they may be different, but the code generation should account for this, and marking every "nullable" field as "optional" confuses the two.

@david-golightly-leapyear
Take a look into avoidOptionals option, I think this is what you  need
https://graphql-code-generator.com/docs/plugins/typescript#avoidoptionals-boolean-default-value-false

@standy I see - avoidOptionals looks to implement the correct behavior. I'm confused why it's not default: true, and indeed why the behavior in this PR would ever be desirable? Unless you're mixing your server & client types inappropriately... but IMO that should be discouraged, it goes against the grain of GraphQL.

@david-golightly-leapyear
That PR fixes inconsistent behavior (described above).

@standy I don't understand how it's inconsistent. The server ("top-level") types and the client ("query") types are often going to be inconsistent in that manner, and that seems to be correct behavior.
The server has to support any possible query, so it can't ever specify the correct type for all queries.
The client does know what fields it's requesting, and is in a position to say "field X is non-optional, because I'm requesting it and it should appear in the GraphQL response".
Indeed, when you look at the behavior of a GraphQL response, any field requested by the client is returned either with a value or null, but cruciall never with undefined or with the field missing.
That's because undefined (aka the ES2019 "optional type") is not a GraphQL type, and the ES2019 "optional type" is also not the same thing as what is called "optional type" in the GraphQL spec.

tl;dr at a minimum, if this PR is going to be preserved, it should also make avoidOptionals default: true, since this PR breaks existing correct usage (avoiding use of undefined in generated client types).

Is this fix available in flow plugins?
seems doesn't work here, or am I missing some config?


Travis makes us work harder now to actually get OpenJDK 6

as per:
https://github.com/travis-ci/travis-ci/issues/8199#issuecomment-327246053

never mind – duplicated in #1219


Connection error

Describe the bug
Hi, when I try the function tdk.gts.search, I got a connection error as below.
Output
http.client.RemoteDisconnected: Remote end closed connection without response
Screenshots

Environment (please complete the following information):

OS: [e.g. Windows 10]
Python Version [e.g. 3.8]

Thank you very much for letting me know. I was able to reproduce the issue. Investigating.

Identified the problem: TDK servers are blocking requests with user agent headers containing the substring Python.

Fix coming shortly.

I guess it can be fixed by using a random agent header as below:
requests.get("https://sozluk.gov.tr/gts?ara=akaryakıt", headers={ "User-Agent" : "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.103 Safari/537.36" })

Yes, but I will implement it using urllib because I don't want to introduce another dependency. (The requests module is not in the standard library.)

The fix is live! Sorry for the inconvenience. Thanks again for reporting!
İyi araştırmalar, kolay gelsin!


Fix the problem of recognition when using different device resolution

The unistroke recognition gives different results depending on the device used.
It seems related to the resolution.

 You want the sample array to be the same size as the template and at same scale.
In recognizedPath:
   // just before doing the best template loop     self.printTemplateSource(sample)
Added a simplified printout to the same file:
   func printTemplateSource(_ points: [CGPoint]) {         var sourceCode = "\nnewTemplate.points = ["         for point in points {             sourceCode += "CGPoint(x:\(point.x), y:\(point.y)),\n"         }         sourceCode += "]"         NSLog("\(sourceCode)")     }

Also, distance function had a bug, here's what it should be:
	func distance(_ point1: CGPoint, point2: CGPoint) -> CGFloat {
		let deltaX = point2.x - point1.x
		let deltaY = point2.y - point1.y

		return sqrt(deltaX * deltaX + deltaY * deltaY)
	}

Thanks again, do you mind opening a PR?


AreaDensity doesn't convert to Mass correctly if unit isn't KilogramsPerSquareMeter

You don't notice this with squants itself because it squants only has the 1 AreaDensity type, but I made a couple custom ones for my app and noticed the issue.
I've submitted a pull request for the fix.https://github.com/garyKeorkunian/squants/pull/115

Thanks.  Sorry I missed this.  I made one simple change request on the PR.  Once that is addressed, I'll merge and publish.


problem with client_id in socket connection registry

I went to annotate some types, but some of the types already there seem to disagree with one another.
this registry dict wants keys to be ints
https://github.com/cpdean/zulip/blob/b985a630bdd93326f32398f63c2ef7e6aeeb4992/zerver/lib/socket.py#L46
this function deletes connections from the dict:
https://github.com/cpdean/zulip/blob/b985a630bdd93326f32398f63c2ef7e6aeeb4992/zerver/lib/socket.py#L61
It uses the client_id on a connection, which is defined as a string here:
https://github.com/cpdean/zulip/blob/b985a630bdd93326f32398f63c2ef7e6aeeb4992/zerver/lib/socket.py#L83
so these three bits disagree.
while i'll probably go ahead and change the existing annotations in this file so things work, it might be worth it to revisit the client_id stuff, could be dead code.
[edited by tabbott]
The most interesting piece is that we can't find any code that actually sets client_id.  Is it possible this is a dead variable?

maybe the type should be Dict[Union[int, str], SocketConnection ? ;)

Yeah that's reasonable to write for now, I think

oh i was kinda trollin but i can do that

This was resolved more than a year ago :)


Could not set destination context: Invalid URL with double url

Hi I'm facing the following error in our production system:
Could not set destination context: Invalid URL: http://api-contentserver:443http://api-contentserver:443/cs/llisapi.dll/api/v1/auth
It is not consequent so sometimes it works, sometimes not. It concerns an Onpremise destination with BA. Note the double mentioned url here.
we are using v1.37.0 there, so my question is: is this probably a sdk issue (which maybe fixed in newer version) or should I look somewhere else?

Hello blaart,
sorry to hear that you have an issue in a productive environment. So from an SDK side we never make calls to a api-contentserver with this dll in the path. Also we have not touched the destination related modules in the last releases to fix something. At which point in the stack does this error appear?
My initial guess would be that this is a response from the destination-service which tries to resolve some auth flow and fails. This would also explain why it works sometimes (destination service ok) and sometimes it fails.  What is the authorization type of the destination? If you switch to some simple thing like basic auth does the error disappear? If so this would be a clear indication that it is the destination service.
Best
Frank

Hi Frank, thanks for your swift response.
The api-contentserver is the url that is configured in the destination. As I cannot reproduce this in a test environment it is hard to say where in the stack this takes place. I'm using Basic Auth as auth type. But I was also thinking of the destination service maybe going wrong somewhere. Not sure, will dive deeper into this. Thanks.

Ah ok, I thought a bit if we manipulate the URL received from the service. But we actually just parse the destination.
So I think the SDK is not changing this but it is the service providing this as a response. You could use the fetchDestinations method directly as a test in your code and log, what is returned by the service and see what is in the URL field.
I hope you can sort out the this flaky issue.

Never mind this, we use an apm module for elastic logging. The logging of that module fooled us.


Unify spelling

This PR supersedes #899 by just Americanize spelling of modeled

I want the bikeshed to be yellow.
(seriously, though, the consistency is fine. I still think the Americans are wrong, but that ship sailed into Boston harbour a long time ago.)

I think this is the wrong PR @kepper :)

Superseded by #1027


in Insert() method, why upbound of updateList is newLevel-1?

Hello, kkdai, here I have a question in skiplist.go Insert() method
https://github.com/kkdai/skiplist/blob/master/skiplist.go#L115
when update the precursor node of the insert node, why the upbound of the for loop is newLevel, since newLevel might be smaller than b.Level.

this might be described in the algorithm


Create integration tests for SSE /v2/events

Currently there are no tests for SSE.
Example test could look like this
package mesosphere.marathon
package integration
import mesosphere.AkkaIntegrationFunTest
import mesosphere.marathon.integration.setup.EmbeddedMarathonTest

@IntegrationTest
class EventStreamIntegrationTest extends AkkaIntegrationFunTest with EmbeddedMarathonTest {
  before(cleanUp())
  test("adding an event subscriber") {
    When("client read event stream")
    val defaultClientEvents = marathon.events("subscribe_event")
    When("second client read event stream with bad filter")
    val filteredClientEvents = marathon.events("non_existing_event_type")
    Then("at list one event should be sent to first subscribers")
    defaultClientEvents.entityString shouldNot be(Seq.empty)
    And("nothing should be sent to second")
    filteredClientEvents.entityString should be(Seq.empty)
  }
}

def events(eventTypes: String*): Seq[MarathonEvent] = ???

Implementation could use https://github.com/hseeberger/akka-sse to parse stream.

Related #4678

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2356. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


[databroker] Add initial notification on subscription

Subscription will yield an initial notification containing current values of everything subscribed to.
Return error if subscription request contains no entries.

I can confirm databroker still works, but as @romainletendart  is working "on the other side" of the new  API , can you give a quick check/thumbs up whether this is working as you expect

I was able to validate the following:

Subscription will yield an initial notification containing current values of everything subscribed to.
Return error if subscription request contains no entries.

However I think we are still missing details of the actual error that occurred.
When a RPC to subscribe() fails, all we get back is a gRPC status code e.g. StatusCode.INVALID_ARGUMENT(3) with details set to "NotFound".
SubscribeResponse does not yet have an Error field which means that only the gRPC error tells us whether something bad happened.
In order to have VISS-compatible error information, I think we would need to have a serialized-as-string Error message behind the 'details' attribute of the gRPC error (see https://github.com/grpc/grpc/tree/master/examples/python/errors).


build: update to latest @nuxt/module-builder

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Not sure if there was a reason this was pinned, but this updates to the latest module builder to avoid an issue with mismatching declarations + compiled files.

I think the problem was the .js extension for runtime modules, where we have some files shared between the client and node side, where .mjs is desired.

Ah, should be good now 👌


Shark plushies now goes rawr on hit.

About the PR

The shark plushies now rawrs when hitting something, or being thrown
Why / Balance

Parity with the lizard plushie
Technical details

A few lines in a .yml file
Media

https://github.com/user-attachments/assets/c12b7e5a-6580-4c99-a846-136719635de5
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl: Minemoder

tweak: The shark plushie now goes rawr when hitting something or being thrown.

P0: Critical

Plushies playing two sounds when thrown should be addressed, although it's a pre-existing issue so I guess we can proceed with this PR regardless

I think it makes a noise when hitting something, and another noise when landing on the ground


Demo database connector doesn't support Embedded Types?

I am using a demo database just to try Prisma and it looks like the demo database is quite restrictive. I don't see any restrictions for Demo Cloud specified here.
Schema
type User {
  email: String! @unique
  id: ID! @id
  name: String!
  type: UserType!
  username: String! @unique
}

type CompanyInfo @embedded {
  name: String!
}

Error
~/g/M/server ❯❯❯ prisma deploy                                                                                ✘ 1
Warning: Your Prisma server and Prisma CLI are currently out of sync. They should be on the same minor version.

Prisma CLI version: prisma/1.34.0 (darwin-x64) node-v10.16.0
Prisma server version: 1.32.0-beta

For further information, please read: http://bit.ly/prisma-cli-server-sync

Deploying service `...` to stage `dev` to server `prisma-us1` 905ms

Errors:

  CompanyInfo
    ✖ The type `CompanyInfo` is marked as embedded but this connector does not support embedded types.

Deployment canceled. Please fix the above errors to continue deploying.
Read more about deployment errors here: https://bit.ly/prisma-force-flag

Demo servers use MySQL which doesn't support embedded type.
Embedded types are only supported by MongoDB so this is expected.


RuntimeError: Given groups=1, weight of size [128, 256, 1, 1], expected input[2, 576, 128, 256] to have 256 channels, but got 576 channels instead

I used train mobilev3 small.yml for training, but I kept reporting errors.
The train mobilev3 small.yml  is from:  https://github.com/ekzhang/fastseg
CUDA_VISIBLE_DEVICES=2,3  python3 -m runx.runx scripts/train_mobilev3small.yml  -i

Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.

None
None
Global Rank: 0 Local Rank: 0
Global Rank: 1 Local Rank: 1
Torch version: 1.6, 1.6.0+cu101
dataset = cityscapes
ignore_label = 255
num_classes = 19
cv split val 2 ['train/monchengladbach', 'train/strasbourg', 'train/stuttgart']
mode val found 655 images
cn num_classes 19
cv split train 2 ['val/lindau', 'val/munster', 'val/frankfurt', 'train/aachen', 'train/bochum', 'train/bremen', 'train/cologne', 'train/darmstadt', 'train/dusseldorf', 'train/erfurt', 'train/hamburg', 'train/hanover', 'train/jena', 'train/krefeld', 'train/tubingen', 'train/ulm', 'train/weimar', 'train/zurich']
mode train found 2820 images
cn num_classes 19
Loading centroid file /app/uniform_centroids/cityscapes_cv2_tile1024.json
Found 19 centroids
Class Uniform Percentage: 0.5
Class Uniform items per Epoch: 2820
cls 0 len 5541
cls 1 len 4897
cls 2 len 5357
cls 3 len 1268
cls 4 len 1537
cls 5 len 5398
cls 6 len 2703
cls 7 len 4610
cls 8 len 5185
cls 9 len 2407
cls 10 len 4436
cls 11 len 3530
cls 12 len 1329
cls 13 len 4864
cls 14 len 415
cls 15 len 398
cls 16 len 183
cls 17 len 551
cls 18 len 2272
Using Cross Entropy Loss
Trunk: mobilenetv3_small
Model params = 1.1M
Selected optimization level O1:  Insert automatic casts around Pytorch functions and Tensor methods.
Defaults for this optimization level are:
enabled                : True
opt_level              : O1
cast_model_type        : None
patch_torch_functions  : True
keep_batchnorm_fp32    : None
master_weights         : None
loss_scale             : dynamic
Processing user overrides (additional kwargs that are not None)...
After processing overrides, optimization options are:
enabled                : True
opt_level              : O1
cast_model_type        : None
patch_torch_functions  : True
keep_batchnorm_fp32    : None
master_weights         : None
loss_scale             : dynamic
Class Uniform Percentage: 0.5
Class Uniform items per Epoch: 2820
cls 0 len 5541
cls 1 len 4897
cls 2 len 5357
cls 3 len 1268
cls 4 len 1537
cls 5 len 5398
cls 6 len 2703
cls 7 len 4610
cls 8 len 5185
cls 9 len 2407
cls 10 len 4436
cls 11 len 3530
cls 12 len 1329
cls 13 len 4864
cls 14 len 415
cls 15 len 398
cls 16 len 183
cls 17 len 551
cls 18 len 2272
Traceback (most recent call last):
File "train.py", line 601, in
main()
File "train.py", line 451, in main
train(train_loader, net, optim, epoch)
File "train.py", line 491, in train
main_loss = net(inputs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/parallel/distributed.py", line 560, in forward
result = self.module(*inputs, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/app/code/semantic-segmentation/network/lraspp.py", line 93, in forward
aspp = self.aspp_conv1(final) * F.interpolate(
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/container.py", line 117, in forward
input = module(input)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/conv.py", line 419, in forward
return self._conv_forward(input, self.weight)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/conv.py", line 416, in _conv_forward
self.padding, self.dilation, self.groups)
File "/usr/local/lib/python3.6/dist-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/amp/wrap.py", line 28, in wrapper
return orig_fn(*new_args, **kwargs)
RuntimeError: Given groups=1, weight of size [128, 256, 1, 1], expected input[2, 576, 128, 256] to have 256 channels, but got 576 channels instead
Traceback (most recent call last):
File "train.py", line 601, in
main()
File "train.py", line 451, in main
train(train_loader, net, optim, epoch)
File "train.py", line 491, in train
main_loss = net(inputs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/parallel/distributed.py", line 560, in forward
result = self.module(*inputs, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/app/code/semantic-segmentation/network/lraspp.py", line 93, in forward
aspp = self.aspp_conv1(final) * F.interpolate(
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/container.py", line 117, in forward
input = module(input)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/conv.py", line 419, in forward
return self._conv_forward(input, self.weight)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/conv.py", line 416, in _conv_forward
self.padding, self.dilation, self.groups)
File "/usr/local/lib/python3.6/dist-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/amp/wrap.py", line 28, in wrapper
return orig_fn(*new_args, **kwargs)
RuntimeError: Given groups=1, weight of size [128, 256, 1, 1], expected input[2, 576, 128, 256] to have 256 channels, but got 576 channels instead
Traceback (most recent call last):
File "/usr/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/usr/local/lib/python3.6/dist-packages/torch/distributed/launch.py", line 261, in
main()
File "/usr/local/lib/python3.6/dist-packages/torch/distributed/launch.py", line 257, in main
cmd=cmd)
subprocess.CalledProcessError: Command '['/usr/bin/python3', '-u', 'train.py', '--local_rank=1', '--dataset', 'cityscapes', '--cv', '2', '--arch', 'lraspp.MobileV3Small', '--max_cu_epoch', '150', '--max_epoch', '175', '--lr', '1e-2', '--lr_schedule', 'poly', '--poly_exp', '1.0', '--syncbn', '--optimizer', 'sgd', '--full_crop_training', '--apex', '--fp16', '--rmi_loss', '--result_dir', 'logs/train_mobilev3small/fastseg-cv2-lraspp.MobileV3Small_cocky-jaguar_2021.07.12_20.20']' returned non-zero exit status 1.

Hi @liuqinglong110, the repository has a couple of small bugs because I ported over the code after leaving Nvidia and did not have the hardware to run it. It's more meant to be a guide. I don't know if you will be able to run it directly.
I believe another person was able to run the code though, here is their branch: https://github.com/yamengxi/semantic-segmentation/commit/338258100a1c38b36d82b3d53bb19ac078165dcf. There are two typos that they corrected in addition to their own configuration stuff:

If you could try that, maybe it would fix the shape mismatch issue you're seeing. I'll push this diff to the repository as well.


Fixes #4437 - Changed indentation on files with 2 spaces to 4 spaces

Description (required)
Fixes #4437
What changes did you make and why?
Used autoformatting in Android Studio on all files that had indentation with 2 spaces instead of 4
Tests performed (required)
None

Ah, would you mind rebasing from master?

Think it should be fixed now, let me know if I need to do anything else. I struggled a bit to figure out how it worked so there might have been an oopsie somewhere

Very sorry, it seems that today's changes made this PR conflict again... would you mind rebasing again?

Should be all good again


Vague error message for invalid or unknown key when adding invalid aws config variables

What happened?
Having aws:XXX variables in your pulumi configurations can result in an undescriptive error:
Diagnostics:
  aws:s3:BucketV2 (xxx):
    error: could not validate provider configuration: 1 error occurred:
        * Invalid or unknown key

Expected Behavior
The error message should state which key is causing the issue.
Steps to reproduce

Create a pulumi project using python
Add any, or all, of the following to your Pulumi.<env>.yaml config:

config:
  aws:account_id: XXX
  aws:access_key_id: XXX
  aws:secret_access_key: XXX

Create a S3 Bucket (or any other AWS resource), for example in main.py:

import pulumi_aws as aws

bucket = aws.s3.Bucket("bucket",
    acl="private",
    tags={
        "Environment": "Dev",
        "Name": "My bucket",
    })

Run pulumi preview or pulumi up, you'll see this error:

Diagnostics:
  aws:s3:BucketV2 (bucket):
    error: could not validate provider configuration: 1 error occurred:
        * Invalid or unknown key
        * Invalid or unknown key
        * Invalid or unknown key

Output of pulumi about
CLI
Version      3.60.0
Go Version   go1.20.2
Go Compiler  gc
Plugins
NAME        VERSION
aws         5.33.0
aws-native  0.49.0
datadog     4.16.0
python      unknown
random      4.12.1
snowflake   0.18.1
Host
OS       darwin
Version  13.1
Arch     arm64
This project is written in python: executable='/Users/xxx/Library/Caches/pypoetry/virtualenvs/xxx-x-infra-O3l4sU9N-py3.11/bin/python3' version='3.11.1
'
Current Stack: seenons/cloud-monitoring/prod
TYPE                                 URN
pulumi:pulumi:Stack                  urn:pulumi:prod::cloud-monitoring::pulumi:pulumi:Stack::cloud-monitoring-prod
pulumi:providers:datadog             urn:pulumi:prod::cloud-monitoring::pulumi:providers:datadog::default_4_16_0
datadog:aws/integration:Integration  urn:pulumi:prod::cloud-monitoring::datadog:aws/integration:Integration::datadog-integration
Found no pending operations associated with seenons/prod
Backend
Name           pulumi.com
URL            https://app.pulumi.com/xxx
User           xxx
Organizations  xxx, xxx
Dependencies:
NAME               VERSION
autopep8           2.0.2
black              22.12.0
flake8             6.0.0
ipython            8.12.0
pip                22.3.1
pulumi-aws         5.33.0
pulumi-aws-native  0.49.0
pulumi-datadog     4.16.0
pulumi-random      4.12.1
pulumi-snowflake   0.18.1
setuptools         65.6.3
toml               0.10.2
wheel              0.38.4
yapf               0.32.0
Note: annonymzed some parts using xxx
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

This should be closed in a recent provider like pulumi-aws v6.31.0 ; please reopen if it is still a problem.


s390: fix build failure with --with-dfltcc-deflate

Patch by Michael Munday (issue #477).

The failure must be in an unrelated flaky test: Ubuntu 18.04 Clang Debug failed with zng_tr_tally: bad match.

I think it would be nice to have an IBM Z machine with GitHub private runner.


allow ember apps to bypass rails pipeline, using EmberCLI asset-rev instead

For production deploys, there is a lot of strength in leveraging EmberCLI's use of broccoli-asset-rev. This approach also embraces the direction where Ember is the primary is front-end technology and, as such, in the primary owner of assets.
The basic approach is that the ember assets (my-app/dist/public/assets) are copied to the ember dist (RAILS_ROOT/public/assets/my-app) such that they go untouched by the Rails/Sprckets asset pipeline. The EmberCli manifest.json is also preserved and is tapped at runtime (after sprockets environment initialization) to update the Rails in-memory asset manifest so it can resolve URLs to traditional asset_path helpers to resolve paths to images or files like my-app/my-app.js.
Seeking to resolve issues like #30, and adapting an approach started by @TeknoFiend in #132, this PR:

adds a new per-app flag (configured in config/ember.rb) bypass_rails_asset_digests which, when set to true enables this behavior
introduces a new class EmberCLI::AssetManifest to load broccoli-asset-rev generated manifests, and inject this data into Sprockets manifests for both Rails 3 and Rails 4
adds rspec for tests
adds a logger to EmberCLI::App so we can get helpful progress messages during the asset compile, similar to what Sprockets does

I haven't tested this fully in a Rails 4 setup yet, but I wanted to get this out there for feedback. If there's interest, I'm happy to pull any fixes or adapt the PR.  Cheers.  :cocktail:

@skylar, did you use
  var app = new EmberApp(defaults, {
    fingerprint: {
      generateRailsManifest: true
    },
    ...
  });

in your ember-cli-build.js/Brocfile.js file for generating the assets or something else?

Not yet working in rails 4.2, tried to use this without success even after fixing the syntax error.

@Samsinite  yes, I used generateRailsManifest in the ember-cli-build.  it would be great if the add-on could set this automatically.

@Samsinite thanks for testing in Rails 4. I fixed the tests. The Rails 4 work was more of a staring point though. The general approach seems right (injecting assets/files into the manifest) but there may be something I've missed. Since there's some interest now I'll create a sample Rails 4 project and take a look.
If there's a fix that stands out to you, lmk.

I agree that the ember addon/rails gem should help setting generateRailsManifest. If there is interest from the folks here, it could be added as a configuration option to both the gem and addon, and be sent across to the addon at asset complication time. Another option for updating the manifest would be modifying the contents of the generated rails manifest file to mirror the ember one with the app path prefixed. Then no in-memory modifications are needed.

@Samsinite My thought was that the in-memory merge of manifests makes more sense because other tools may need to look at the raw rails asset manifest that is generated.   Also, if rails-cli-ember is disable or not loaded in a certain environment, then you would not want the manifest changed.  For that reason, this PR "imports" the ember manifest into rails during app load, modifying the rails runtime to be ember-cli savvy (which seems to be the major approach of ember-cli-rails).
If you can post the broccoli-generated manifest and the merged manifest from your Gist, I bet we can fix up the rails 4 on the merge easily.  (i.e., it gives us a test case currently missing in the PR right now)

@Samsinite looking at:
http://edgeguides.rubyonrails.org/asset_pipeline.html
it seems that my files index for Rails 4  was using the logical file path, not the fingerprinted path as in your gist at https://gist.github.com/Samsinite/632e99134b43e96d7162.
Do you have a sample rails4 / ember-cli app in which I can test this?

I don't, I would have to put together a sample app.

Also, I'm not super familiar with rails gem configuration defaults, but looks like it might need a attr_accessor : bypass_rails_asset_digests  in the ember-cli-rails/lib/ember-cli/configuration.rb file?

Nevermind, I see you exposed it as an option passed to the app:
EmberCLI.configure do |c|
  c.app :admin_panel, bypass_rails_asset_digests: true
end

We're sorry that this issue has sat for so long.
The project is now maintained by thoughtbot, and we're declaring an issue bankruptcy of sorts.
This looks promising, and we're really interested fixing this problem. However, since this project lacks a test suite (and CI in general), we're going to hold off on merging PRs until we've added some coverage.
Thanks for your patience.

Sounds awesome, thanks Sean.

@skylar and co.
Thanks for your help. Please read https://github.com/thoughtbot/ember-cli-rails/issues/30#issuecomment-154768936 and follow up with how it works out.

I'll be closing this PR until we sort out https://github.com/thoughtbot/ember-cli-rails/issues/30#issuecomment-154768936.
Please leave comments in that issue thread.


[tutorial] error with max() while running SpikeSorting.populate()

Still in the tutorial notebook 1_spikesorting.ipynb, and issues #41 and #40 fixed. The SpikeSortingParameters are successfully inserted, and (SpikeSortingParameters & {'nwb_file_name' : nwb_file_name2}) has my entry in it.
When I run this cell to populate
# Specify entry (otherwise runs everything in SpikeSortingParameters)
# `proj` gives you primary key
SpikeSorting.populate([(SpikeSortingParameters & {'nwb_file_name' : nwb_file_name2}).proj()])

The output and error:
Elapsed time for creating analysis NWB file: 0.3789808750152588 sec

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-61-1dd950092fb2> in <module>
      1 # Specify entry (otherwise runs everything in SpikeSortingParameters)
      2 # `proj` gives you primary key
----> 3 SpikeSorting.populate([(SpikeSortingParameters & {'nwb_file_name' : nwb_file_name2}).proj()])

~/anaconda3/envs/nwb_datajoint6/lib/python3.8/site-packages/datajoint/autopopulate.py in populate(self, suppress_errors, return_exception_objects, reserve_jobs, order, limit, max_calls, display_progress, *restrictions)
    151                     self.__class__._allow_insert = True
    152                     try:
--> 153                         make(dict(key))
    154                     except (KeyboardInterrupt, SystemExit, Exception) as error:
    155                         try:

~/proj/nwb_datajoint/src/nwb_datajoint/common/common_spikesorting.py in make(self, key)
    500             # Create a new NWB file for holding the results of analysis (e.g. spike sorting).
    501             # Save the name to the 'key' dict to use later.
--> 502             key['analysis_file_name'] = AnalysisNwbfile().create(key['nwb_file_name'])
    503
    504         sort_interval =  (SortInterval & {'nwb_file_name': key['nwb_file_name'],

~/proj/nwb_datajoint/src/nwb_datajoint/common/common_nwbfile.py in create(self, nwb_file_name)
    136                             nwb_object.pop(module)
    137
--> 138             analysis_file_name = self.__get_new_file_name(nwb_file_name)
    139             # write the new file
    140             print(f'Writing new NWB file {analysis_file_name}')

~/proj/nwb_datajoint/src/nwb_datajoint/common/common_nwbfile.py in __get_new_file_name(cls, nwb_file_name)
    151         names = (AnalysisNwbfile() & {'nwb_file_name': nwb_file_name}).fetch('analysis_file_name')
    152         n1 = [str.replace(name, os.path.splitext(nwb_file_name)[0], '') for name in names]
--> 153         max_analysis_file_num = max([int(str.replace(ext, '.nwb', '')) for ext in n1])
    154         # name the file, adding the number of files with preceeding zeros
    155         analysis_file_name = os.path.splitext(nwb_file_name)[0] + str(max_analysis_file_num+1).zfill(6) + '.nwb'

ValueError: max() arg is an empty sequence

Sorry, that’s a bug. If you have time to fix it, you could add a “try” statement before that line with
except:
max_analysis_file_num = 0

On Jun 9, 2021, at 12:58 PM, Ji Hyun Bak @.***> wrote:
Still in the tutorial notebook 1_spikesorting.ipynb, and issues #41 https://github.com/LorenFrankLab/nwb_datajoint/issues/41 and #40 https://github.com/LorenFrankLab/nwb_datajoint/issues/40 fixed. The SpikeSortingParameters are successfully inserted, and (SpikeSortingParameters & {'nwb_file_name' : nwb_file_name2}) has my entry in it.
When I run this cell to populate
Specify entry (otherwise runs everything in SpikeSortingParameters)
proj gives you primary key
SpikeSorting.populate([(SpikeSortingParameters & {'nwb_file_name' : nwb_file_name2}).proj()])
The output and error:
Elapsed time for creating analysis NWB file: 0.3789808750152588 sec
ValueError                                Traceback (most recent call last)
 in
1 # Specify entry (otherwise runs everything in SpikeSortingParameters)
2 # proj gives you primary key
----> 3 SpikeSorting.populate([(SpikeSortingParameters & {'nwb_file_name' : nwb_file_name2}).proj()])
~/anaconda3/envs/nwb_datajoint6/lib/python3.8/site-packages/datajoint/autopopulate.py in populate(self, suppress_errors, return_exception_objects, reserve_jobs, order, limit, max_calls, display_progress, *restrictions)
151                     self.class._allow_insert = True
152                     try:
--> 153                         make(dict(key))
154                     except (KeyboardInterrupt, SystemExit, Exception) as error:
155                         try:
~/proj/nwb_datajoint/src/nwb_datajoint/common/common_spikesorting.py in make(self, key)
500             # Create a new NWB file for holding the results of analysis (e.g. spike sorting).
501             # Save the name to the 'key' dict to use later.
--> 502             key['analysis_file_name'] = AnalysisNwbfile().create(key['nwb_file_name'])
503
504         sort_interval =  (SortInterval & {'nwb_file_name': key['nwb_file_name'],
~/proj/nwb_datajoint/src/nwb_datajoint/common/common_nwbfile.py in create(self, nwb_file_name)
136                             nwb_object.pop(module)
137
--> 138             analysis_file_name = self.__get_new_file_name(nwb_file_name)
139             # write the new file
140             print(f'Writing new NWB file {analysis_file_name}')
~/proj/nwb_datajoint/src/nwb_datajoint/common/common_nwbfile.py in __get_new_file_name(cls, nwb_file_name)
151         names = (AnalysisNwbfile() & {'nwb_file_name': nwb_file_name}).fetch('analysis_file_name')
152         n1 = [str.replace(name, os.path.splitext(nwb_file_name)[0], '') for name in names]
--> 153         max_analysis_file_num = max([int(str.replace(ext, '.nwb', '')) for ext in n1])
154         # name the file, adding the number of files with preceeding zeros
155         analysis_file_name = os.path.splitext(nwb_file_name)[0] + str(max_analysis_file_num+1).zfill(6) + '.nwb'
ValueError: max() arg is an empty sequence
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/LorenFrankLab/nwb_datajoint/issues/42, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV4PSKI4RDXNMZVAHY6SQDTR7BWLANCNFSM46MXAR4Q.

Thanks Loren, I can add that!

OK, I was able to fix the bug above, but now got another error.
Still running this cell
# Specify entry (otherwise runs everything in SpikeSortingParameters)
# `proj` gives you primary key
SpikeSorting.populate([(SpikeSortingParameters & {'nwb_file_name' : nwb_file_name2}).proj()])

These are the output and error messages now:
beans20190718_jhbak_000001.nwb
Writing new NWB file beans20190718_jhbak_000001.nwb
Elapsed time for creating analysis NWB file: 1.36918044090271 sec
Elapsed time for filtered recording extractor setup: 3.345864772796631 sec
Elapsed time for NWB recording extractor create from file: 3.2766377925872803 sec
Elapsed time for writing filtered NWB recording extractor to /stelmo/nwb/spikesorting/beans20190718_jhbak_000001.nwb/beans20190718_jhbak_.nwb_beans_02_r1_10s_8_mountainsort4_beans.nwb: 2.1219253540039062e-05 sec

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-44-1dd950092fb2> in <module>
      1 # Specify entry (otherwise runs everything in SpikeSortingParameters)
      2 # `proj` gives you primary key
----> 3 SpikeSorting.populate([(SpikeSortingParameters & {'nwb_file_name' : nwb_file_name2}).proj()])

~/anaconda3/envs/nwb_datajoint6/lib/python3.8/site-packages/datajoint/autopopulate.py in populate(self, suppress_errors, return_exception_objects, reserve_jobs, order, limit, max_calls, display_progress, *restrictions)
    151                     self.__class__._allow_insert = True
    152                     try:
--> 153                         make(dict(key))
    154                     except (KeyboardInterrupt, SystemExit, Exception) as error:
    155                         try:

~/proj/nwb_datajoint/src/nwb_datajoint/common/common_spikesorting.py in make(self, key)
    543         with Timer(label=f'writing filtered NWB recording extractor to {extractor_nwb_path}', verbose=True):
    544             # TODO: save timestamps together
--> 545             se.NwbRecordingExtractor.write_recording(recording, save_path=extractor_nwb_path,
    546                                                     buffer_mb=10000, overwrite=True, metadata=metadata,
    547                                                     es_key='ElectricalSeries')

TypeError: write_recording() got an unexpected keyword argument 'es_key'

This ^ has to do with spikeextractors version. Keyword argument es_key was introduced after the latest released tag (0.9.6) - we need local install at this time.

Or perhaps we could point to a specific spikeextractors commit from our environment.yaml


Setup Kanban Board for the Final Capstone

Hi :wave:
For this project, we are 3:

@Daron976
@vflorin1995
@laiifuu

Screenshot of the Kanban:

Here's the link to the Project:
https://github.com/users/laiifuu/projects/4/views/1

Hi @laiifuu,
Wow, you did it 🎉

Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
To highlight

All tasks have been pointed✔
The paintings are fair, and equal✔
All tasks have descriptions✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Add gamma initialization and se module for gluon resnet model

Description
For resnet v1 and v2

Suqeeze-and-Excitation module is added, with new models having 'se_' prefix.
Add option to initialize the gamma in the last BN layer of each block of resnet to zero

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
[X] Code is well-documented:
[X] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@hetong007 bouncing this.

I'm closing this for now, as it hasn't been trained to offer a pretrained model yet thus not ready.


Tpl

Hello @aggelosgkiokas! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file trompace/connection.py:

Line 76:1: E302 expected 2 blank lines, found 1
Line 88:35: W292 no newline at end of file

In the file trompace/queries/controlaction.py:

Line 42:1: E302 expected 2 blank lines, found 1
Line 51:20: W292 no newline at end of file


icons not appearing on self service UI

Description of problem:
After a custom skin is build and uploaded all the icons like the shoppingcart, notification icon, question mark, etc are missing from the interface.
How reproducible:
Just get the current code from github and build it with yarn.
Steps to Reproduce:
1.
2.
3.
Actual results:
no icons being in the navbar etc.
Expected results:
icons being in the navbar is expected
Additional info:

Confirmed, great catch @jweewer !!!


Error "save_training_settings_to_txt" on training

Latest repo
sd folder removed as a fresh start
created a new embedding
start training
Textual inversion embeddings loaded(1): RocksyLight
Training at rate of 0.005 until step 100000
Preparing dataset...
100% 14/14 [00:16<00:00,  1.20s/it]
Applying xformers cross attention optimization.
Error completing request
Arguments: ('RocksyLight', '0.005', 1, 1, '/content/gdrive/MyDrive/Processed/RocksyLight', 'textual_inversion', 512, 512, 100000, 'disabled', '0.1', False, 0, 'once', 500, 500, '/content/gdrive/MyDrive/sd/stable-diffusion-webui/textual_inversion_templates/style_filewords.txt', True, False, '', '', 20, 0, 7, -1.0, 512, 512) {}
Traceback (most recent call last):
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/call_queue.py", line 45, in f
    res = list(func(*args, **kwargs))
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/call_queue.py", line 28, in f
    res = func(*args, **kwargs)
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/textual_inversion/ui.py", line 33, in train_embedding
    embedding, filename = modules.textual_inversion.textual_inversion.train_embedding(*args)
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/textual_inversion/textual_inversion.py", line 315, in train_embedding
    if shared.opts.save_training_settings_to_txt:
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/shared.py", line 489, in __getattr__
    return super(Options, self).__getattribute__(item)
AttributeError: 'Options' object has no attribute 'save_training_settings_to_txt'

That must be an issue with the A1111 repo, I'm sure they will fix it soon

I'm having two colabs opened now and one is traing while this one still has the same error.

Just tested and I found this particularly not working even I removed everything from th gdrive and re-uploaded. Weird.


Question about of generate novel view animations

When I know the R and T of the current view, how do I generate the motion trajectory of the new view on the kitti dataset to generate the demo, can you provide the code?
How to get this file of "./scripts/videos/trajectories/simple_movement.npy"?

Hi!
Sorry for the late reply!
I generate these camera trajectories with blender. You can use this example script within blender to export a trajectory:
scripts/blender/export_camera_trj.py
Best,
Felix


[Idea]: Task Scheduling

Idea Title
Task Scheduling
Idea Description
Task Scheduling algorithm using Topological Sorting (Kahn’s Algorithm) implemented in C. This algorithm determines an order in which tasks can be completed given dependencies between them.
Potential Benefits

[ ] Improves user experience
[X] Enhances learning
[X] Adds a new feature
[ ] Improves code quality
[X] Other

Implementation Suggestions (Optional)
Stack, queues

@ajay-dhangar I have raised a PR #1549 to fix this issue


Add GitHub workflow for monolith and recommendation service

This pull request introduces two new GitHub Actions workflows to automate the build and deployment processes for the monolith and recommendation service to Azure Kubernetes Service (AKS).
New Workflows for Build and Deployment:

Monolith Service:

Added a workflow to build and deploy the monolith service to AKS on pushes to the main branch and pull requests to the github-workflow branch. (.github/workflows/deploy-monolith.yml)

Recommendation Service:

Added a workflow to build and deploy the recommendation service to AKS on pushes to the main branch and pull requests to the github-workflow branch. (.github/workflows/deploy-recommendation-service.yml)

Redeploy. Fixed on Azure side


Replace muParserX with Expression Toolkit to resolve segfaults

MuparserX was causing segfaults due to it's terrible use of singletons (the most antiest of patterns)
Swap to using another library without this problem.

Superseded by another PR


Add documentation in the README

Just the last commit, it's a chained PR.
If there is something we should change of the behaviour of kallax before v1, now would be the time to do so, so please, read carefully all the documentation and point out things you'd change before this makes it to the v1.
To read it formatted: https://github.com/erizocosmico/go-kallax/blob/docs/readme/README.md

Codecov Report

Merging #86 into master will not impact coverage by -1.38%.

@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
- Coverage    77.8%   76.43%   -1.38%
==========================================
  Files          15       15
  Lines        2244     2236       -8
==========================================
- Hits         1746     1709      -37
- Misses        332      368      +36
+ Partials      166      159       -7

Impacted Files
Coverage Δ

query.go
92.02% <ø> (ø)
:white_check_mark:

model.go
85.45% <ø> (ø)
:white_check_mark:

generator/generator.go
0% <ø> (ø)
:white_check_mark:

timestamps.go
100% <100%> (ø)
:white_check_mark:

resultset.go
76.27% <100%> (ø)
:white_check_mark:

types/types.go
74.21% <44.44%> (-0.54%)
:x:

batcher.go
77.57% <50%> (ø)
:white_check_mark:

types/slices.go
61.99% <62.16%> (ø)
:white_check_mark:

generator/processor.go
78.54% <ø> (-11.75%)
:x:

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 343a08c...45753a9. Read the comment docs.

So, if we wanna release this to the public and not force them to use our UUIDs (which are actually ULIDs). Should we make the modification to allow primary keys of arbitrary types (simple ones, not composite keys) before v1.0. Because that would be a breaking change and we'd have to bump to v2.0 when the change is implemented.
@smola @ajnavarro @Serabe @dpordomingo @mcuadros

Done @ajnavarro

@erizocosmico As we discussed offline, we have not defined any IDs other than UUID/ULID so far, but defining our own for some future cases does not sound far fetched.

@smola yes, there is a PR for that (https://github.com/src-d/go-kallax/pull/98) and we'll update the docs once it's merged and the api for it is final. Right now, it is kind of a WIP to see if we like the solution implemented or not, so we're waiting for that before updating these docs

@dpordomingo I'm waiting for the custom pks thing to be merged before updating the docs in case something changes.

This documentation is actually outdated. Take into account the new way to implement the ID: ID           kallax.ULID pk:""

I'd add a caveats section, warning about the time aliases, and identifier aliases
Could it be added the benchmark you made comparing kallax with gorm?
I'd ad a logo or whatever image, random examples:

whatever on your own ;)

The benchmark would need a little bit of work before being added here. And so would the logo haha. But yeah, we definitely need one.

@dpordomingo changes made

@dpordomingo where are the //TO CONSIDER comments? I can't find them

Fixed @dpordomingo

@dpordomingo added docs for findbys

Fixed @dpordomingo


"fair" cpu sharing for jobs of different users, fuzzy worker-queues

We have different users, for which we do several time-intensive background-jobs, like scaling a large numbers of Images. The standard strategy is "first in, first out", what can be frustration for a user with just 10 Images, uploaded just after an other user with thousands. So I introduced the "balance_id", which can just be set to the user_id for example.
Additionally I let the workers choose the defined queus to work of by mysql-"LIKE '%queue%', so one can easily define a groub of ques for a worker (eg. '_import', which will do image_import and document_import)

Coverage decreased (-2.04%) to 87.37% when pulling f846ea1a905bdbf9cf43fc42143263034c3b0080 on philister:master into 270b06eef0879a7ea167eb41182336aa14fb762e on collectiveidea:master.

Coverage decreased (-2.04%) to 87.37% when pulling f846ea1a905bdbf9cf43fc42143263034c3b0080 on philister:master into 270b06eef0879a7ea167eb41182336aa14fb762e on collectiveidea:master.

Coverage decreased (-2.04%) to 87.37% when pulling f846ea1a905bdbf9cf43fc42143263034c3b0080 on philister:master into 270b06eef0879a7ea167eb41182336aa14fb762e on collectiveidea:master.

Coverage decreased (-2.04%) to 87.37% when pulling f846ea1a905bdbf9cf43fc42143263034c3b0080 on philister:master into 270b06eef0879a7ea167eb41182336aa14fb762e on collectiveidea:master.

You can get similar behavior by playing with priority when creating a job. There's a bunch of ways to approach it but one way would be to increment priority as the user created more jobs in a batch. So a user's first job runs with priority 1, their second with priority 2, etc.
I am not in favor of accepting this feature, or others like it, that try to inspect the state of the queue.


Crash after update

Crash after update.
Minecraft: 1.12.2
Forge: <IP_ADDRESS>39
Versions (new) - crash:
Refined Storage 1.6+Refined Storage Addons 0.4
Versions (old) - very good work:
Refined Storage 1.5.34+Refined Storage Addons 0.3
Crash report - https://gist.github.com/kellixon/c0388ca5cd0bc04ddf6602a59fa9ce7f

Problem in storage tech


Faster unban command a better regex.

We don't need any commits or merges from you.
Thanks Linker.


SAMZA-1696: Fix TestKeyValueStores flaky test

Fix flaky test testParallelReadWriteSameKey that was failing due to a race condition between the main test thread and the thread manipulating the key-value store under test. Specifically, the main test thread could assert the store has received a value before the other thread gets to set it. The hard-coded wait durations did not guarantee the main test thread would wait for all other threads to complete, causing it to assert prematurely.

@xinyuiscool Could you, please, review this change when you get a chance?


Update monitor-instances-health-check.md

@msangapu-msft
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@msangapu-msft
Can you review this old PR and determine whether it needs to be closed or merged?
@MicrosoftDocs/public-repo-pr-review-team


Fix coverage badge

Trivial, so merging when travis finishes

Pull Request Test Coverage Report for Build 257

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.382%

Totals

Change from base Build 254:
0.0%

Covered Lines:
1153

Relevant Lines:
1184

💛  - Coveralls


STELLAR-3687 Auto close stream

Think this would address https://github.com/infostellarinc/stellarcli/issues/122
@rockets3600 could you explain the usage and expected behaviour in the PR description? Thanks!

@chelliwell We're curious about what you think concerning the flag usage for this. Is it okay to use the default values for delay and close time if they are not provided? Or would you prefer that if auto-close is on, the delay and close time must be set explicitly. Thanks!

Also curious about the layout of params
--enable-auto-close true --auto-close-delay duration 5s --auto-close-time 2006-01-02'T'15:04:05Z

vs
--auto-close 5s 2006-01-02'T'15:04:05Z

@chelliwell We're curious about what you think concerning the flag usage for this. Is it okay to use the default values for delay and close time if they are not provided? Or would you prefer that if auto-close is on, the delay and close time must be set explicitly. Thanks!

I see that you've edited the top message in this thread. Is that now the up-to-date description of the parameters?

@chelliwell We're curious about what you think concerning the flag usage for this. Is it okay to use the default values for delay and close time if they are not provided? Or would you prefer that if auto-close is on, the delay and close time must be set explicitly. Thanks!

I see that you've edited the top message in this thread. Is that now the up-to-date description of the parameters?

The description is a bit outdated right now. I'll update it once things are more settled and let you know that I've done so.

Tested several times with auto close time set before and after the expected end of data transmission and saw good results.
One thing I didn't see documented anywhere is the behaviour of when the auto close time is set after the end of expected transmission. In my tests the stream closed 10s after the provided time, but it wasn't immediately obvious where the 10s came from.

To clarify my last comment, the stream is closed at the provided timestamp + specified duration + stream latency. In my example I used a 5s duration and stream latency was ~5s so the stream was closed after 10 seconds. We can look into this duration at a later date.


General Preferences - Standardise download location path

This PR fixes the cosmetic issue of the icon in General Preferences being a generic icon after a default install of Vienna.

The cause of this was the default download location being set to ~/Desktop which caused [[NSWorkspace sharedWorkspace] iconForFile:downloadFolderPath]; to not be able to find a real icon.
The path needed to be expanded so I chose to use stringByStandardizingPath instead of only stringByExpandingTildeInPath because stringByStandardizingPath expands the tilde as well as doing some other clean up. We rarely call this so the extra functionality provided by stringByStandardizingPath won't cause any performance impact.

I chose to try to correct any path that was in the preferences instead of changing the initial path from ~/Desktop because we don't know what value a user might try to this to using scripts etc and this might prevent problems in the future.

Keep in mind that if we ever support sandboxing, ~/Desktop won't be available to us without prior user consent. In addition, we should really use NSFileManager -URLForDirectory: and -URLsForDirectory: to resolve these paths.

OK cool, I'll have a look at changing it.

OK this is ready for review again.

This still is not working for me. The new path is now:

file:/Users/user/Desktop

Shouldn't we also change the default location to the Downloads folder ?

@barijaona: I concur (as I wrote above).

@josh64x2 Will you continue to work on this or should we drop it?

If I haven't fixed it in a month we should delete it :)


Added Group Column tests

@jhoneill - first pass at some tests. Thoughts?

Sorry I didn't have a chance to look over them - madness on all sides at the moment!


Make the app explicitly ask for permission to update!

The version 12.3.0 had introduced breaking changes that made my keyboard completely unusable.
That happened suddenly and while I was working because the app auto-updated to the new version without my permission.
I would like to see the default behavior of the app changed so that it NEVER updates without the user's explicit permission to do so.
It is a huge disruption when suddenly, during a normal workday, the app suddenly breaks the entire keyboard layout making me unable to do any work and forcing me to spend hours on troubleshooting the issue.
So, the updates should only happen when I explicitly confirm that it's OK to update. Never automatically. I'd like to log this as a feature request. Thanks.

I agree with this. But I guess on my case I might've clicked yes on an update prompt but then after it, I was unable to use my capslock as an esc/ctrl combo. I had to revert back to 12.2.0 just so I could have it back again. Not sure what I should change with my karabiner.json file so that I can upgrade to 12.3.0 safely.

Version 12.3.0 asks if you want to update to 12.4.0.


Fixed Positioning Floating Header to the Table, Then the Window

Greetings.
I work for a company which exclusively uses IE (currently v.11), so I'm developing for that browser specifically.
For simplicity's sake, I'll call table1 the original table ($table).
I'll call the cloned/duplicated/copied thead table table2  (contains the thead content after being initialized).
I've set the useAbsolutePositioning property to false.  So, I'm relying on fixed positioning.
Is there any way to dynamically switch between fixing table2 to the top of table1 and then fixing table2 to the top of the window?
It seems in IE 11 that as table2 has to adjust to being a variable distance from the top of the window (I mean, the initial scrolling until the top of the window meets the top of the table), table2 noticeably shifts/skips/jumps until it has a top:0px; style.
Is there any way to switch between the two points of reference (table1 and the window) for fixed positioning so that IE doesn't have to keep (noticeably) changing the top position style of table2?
I hope I've worded that adequately well.
Here's a fiddle: https://jsfiddle.net/p6SRV/106/
Thanks for your help.

The frustrating thing is that Chrome doesn't jump/skip/shift with this configuration.  It's just IE that handles it poorly.  :/

This was a big problem in IE8, but I didnt think it was a big deal in 11.
see: https://github.com/mkoryak/floatThead/blob/master/jquery.floatThead.js#L254
might be able to change that to true and see if helps you, then i can make a change to expose it

In my opinion, the minor lag isn't a big deal.
But, I have a customer who really would like the thead stuck to the table, and then stuck to the top of the window, and to scroll with the rest of the page's content instead of appearing to lag a split second behind each scroll.
Here's a fiddle with both the jquery.floatThead.js and jquery.floatThead._.js files included in the HTML section in a

Oops.   I guess I can't use html tags.
https://jsfiddle.net/p6SRV/109/
I changed the line (262) that you were suggesting.

there is currently a bug where padding on the table or scroll container makes fixed positioning not work properly. your fiddle seems to have this problem for me on chrome, and probably ie11. if you can get around it via css then you can do this:
https://jsfiddle.net/p6SRV/111/
does that look better in ie11? if it doesnt actually work how you want it, then you can try listening to the floated event and destroy the table and reinit it with the positioning you require:
(didnt actually run this code)
$table.on("floatThead", function(e, isFloated, $floatContainer){
    if(isFloated){
        $(this).floatThead("destroy").floatThead({....}).trigger("reflow");
    } else {
        $(this).floatThead("destroy").floatThead({....}).trigger("reflow");
    }
});

I did a bit of IE researching and found that by unchecking IE's "Use smooth scrolling" setting, this issue no longer exists.  You can do this by clicking the settings cog/gear -> Internet Options -> Advanced (tab) -> Browsing  -> "Use smooth scrolling" (uncheck).
I did remove the ieVersion <= 9 condition from this line.
var absoluteToFixedOnScroll = {ieVersion <= 9 &&} !locked && useAbsolutePositioning;

And I initialized the floatThead like this:
$table.floatThead({
	useAbsolutePositioning: false
});

I think this is how I'm going to move forward with this project.
Thanks for your help!

sounds good, ill make that be an option you can set for the next version

wait, if you init with useAbsolutePositioning: false then absoluteToFixedOnScroll ends up as false, so maybe you only need to do the "Use smooth scrolling" (uncheck) part?

Perhaps.  I didn't test it with useAbsolutePositioning: true, so I'm not sure if it's still an issue there.

Not to bring back up this sore subject, but this seems to happen on Edge, now, too. However, I don't really see and option to uncheck smoothing.

there is nothing I can do about this as far as I can tell. Does editing the source and changing absoluteToFixedOnScroll to be always true help with anything?

No. That does not help. I've even attempted to change the setting in your code to:
if (pos != null && (oldTop != pos.top || oldLeft != pos.left)) {                               if (ieVersion === 8 || ieVersion === 9 || ieVersion === 10 || ieVersion === 11 || /Edge\/\d./i.test(navigator.userAgent)) {               $floatContainer.css({                 top: pos.top,                 left: pos.left               });             } else {                             var transform = 'translateX(' + pos.left + 'px) translateY(' + pos.top + 'px)';               $floatContainer.css({                 '-webkit-transform' : transform,                 '-moz-transform'    : transform,                 '-ms-transform'     : transform,                 '-o-transform'      : transform,                 'transform'         : transform,                 'top': 0,                 'left': 0               });             }
if you watch edge as you scroll, it's very 'notchy' (forgive me if I'm not using the right term). It will jump several numbers at once in the pos.top.

sounds like this is a bug i cannot fix, it has to do with the speed of the
javascript engine / cpu
On Fri, Oct 14, 2016 at 4:57 PM, dallaslopez<EMAIL_ADDRESS>wrote:

No. That does not help. I've even attempted to change the setting in your
code to:
if (pos != null && (oldTop != pos.top || oldLeft != pos.left)) {
if (ieVersion === 8 || ieVersion === 9 || ieVersion === 10 || ieVersion
=== 11 || /Edge/\d./i.test(navigator.userAgent)) {
$floatContainer.css({
top: pos.top,
left: pos.left
});
} else {
var transform = 'translateX(' + pos.left + 'px) translateY(' + pos.top +
'px)';
$floatContainer.css({
'-webkit-transform' : transform,
'-moz-transform' : transform,
'-ms-transform' : transform,
'-o-transform' : transform,
'transform' : transform,
'top': 0,
'left': 0
});
}
if you watch edge as you scroll, it's very 'notchy' (forgive me if I'm not
using the right term). It will jump several numbers at once in the pos.top.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/mkoryak/floatThead/issues/233#issuecomment-253917359,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXbSVkFVj3OIn7CCAVM0Rri1khl1Mq8ks5qz-y8gaJpZM4FrF-S
.

What's ironic is that it's not happening in Chrome or Firefox.

Why is that ironic? Things have never worked right in ie


Correction for raw affiliation PHIM Plant Health Institute, University of Montpellier, CIRAD, INRAE, Institut Agro, IRD, 34090 Montpellier, France; UMR PHIM - Plant Health Institute of Montpellier (CIRAD Campus International de Baillarguet TA A-54/K
34398 Montpellier

France)

Correction needed for raw affiliation PHIM Plant Health Institute, University of Montpellier, CIRAD, INRAE, Institut Agro, IRD, 34090 Montpellier, France; UMR PHIM - Plant Health Institute of Montpellier (CIRAD Campus International de Baillarguet TA A-54/K
34398 Montpellier

France)
raw_affiliation_name: PHIM Plant Health Institute, University of Montpellier, CIRAD, INRAE, Institut Agro, IRD, 34090 Montpellier, France; UMR PHIM - Plant Health Institute of Montpellier (CIRAD Campus International de Baillarguet TA A-54/K
34398 Montpellier
France)
new_rors: 03fkrcm40;05kpkpg04;051escj72;003vg9w96;05y5s3h28
previous_rors: 03fkrcm40;05kpkpg04;051escj72;003vg9w96
works_examples: W3157111940
contact: 01b5cbf94ada201b6f2de4c7010897be:466c5f1f34b678c72bca561326b0e3e1a958 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Credential list not updating when credential revocation notification received

If on credential list view and revocation notification received, the list of credentials are not updated to reflect the revoked credentials. Even after navigating away from the view and back does not reflect the revocation. Only after closing and reopening the wallet does the list display the revocation status.

I'll take this one but github isn't allowing me to assign myself


 The result of FnApi PostCommit can be found in  https://builds.apache.org/job/beam_PostCommit_Java_FnApi_GradleBuild_PR/3/

Run Seed Job

Run Java PortabilityApi PostCommit

Renamed FnApi to PortaboilityApi and all tests passed. @lukecwik could you please take another look at?


use FirebaseMessaging instead of FirebaseInstanceId to get token

I'm basically copying this fix https://github.com/zo0r/react-native-push-notification/commit/66243dd200efed95d72d2590ab3b0bf47fe22537
That commit was included in today's release of 7.3.0 but we are quite far behind that version so I just want to change as little as possible while getting our android build working again.

Got a notification for this, didn't know I was still subscribed :D
Thanks for working on this @tplorts!

forgot to include https://github.com/Truebill/react-native-push-notification/pull/1/commits/8b9ba9192c3cd5158dd4c10237daf1ff8ad3e2ea because I had written it in truebill-native/node_modules/react-native-push-notification/android/src/main/java/com/dieam/reactnativepushnotification/modules/RNPushNotificationRegistrationService.java -- to make testing quick I just copied the updated file directly into that and then realized I had that missing import.

yolo!


Cannot scroll large number of explorers

testing #48407
When contributing a large number of explorers, at some point you cannot scroll to them.  for example, i created 20+ explorers:

When I start to expand each one, such that the extend pass the bottom of the viewlet, there is no scroll bar that would let me scroll down to see the explorers that are below the screen.

This isn't specific to #48407, the same thing happens if I contribute alot of explorers to the main explorer.

Also adding @joaomoreno as this is related to split view.

Splitview doesn't really support this, since there would be scrollbars within scrollbars... But we can think about it.


When Sandstorm starts and has a https certificate created by the installer, it can start a https daemon

This requires some monkeypatching.

This is basically done. It requires manually setting the HTTPS_PORT in sandstorm.conf and requires https://github.com/sandstorm-io/sandstorm/pull/900 be merged.
I've renamed the ticket to indicate that an opt-in HTTPS_PORT is enough for success here.

This is now done & merged, wowee!


"no such table: constance_config" when running test suite

Describe the problem
Recently, I upgraded constance from 3.1.0 to 4.0.1 in my Django project, which has gone fine. I ran the migrations and we were all set. Functionally, we are all good: it works like it did before, no issues.
However, when running my test command python manage.py test, I started to get this error from Django constance.
➜  poetry run python manage.py test

Found 452 test(s).
Creating test database for alias 'default'...
copy data from old constance table to a new one
Traceback (most recent call last):

copy data from old constance table to a new one
Traceback (most recent call last):
  File "/Users/paul/Library/Caches/pypoetry/virtualenvs/bloom-nofos-ztRUj9S3-py3.11/lib/python3.11/site-packages/django/db/backends/utils.py", line 89, in _execute
    return self.cursor.execute(sql, params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/paul/Library/Caches/pypoetry/virtualenvs/bloom-nofos-ztRUj9S3-py3.11/lib/python3.11/site-packages/django/db/backends/sqlite3/base.py", line 328, in execute
    return super().execute(query, params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
sqlite3.OperationalError: no such table: constance_config

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/paul/Library/Caches/pypoetry/virtualenvs/bloom-nofos-ztRUj9S3-py3.11/lib/python3.11/site-packages/constance/migrations/0002_migrate_from_old_table.py", line 19, in _migrate_from_old_table
    cursor.execute(
  File "/Users/paul/Library/Caches/pypoetry/virtualenvs/bloom-nofos-ztRUj9S3-py3.11/lib/python3.11/site-packages/django/db/backends/utils.py", line 67, in execute
    return self._execute_with_wrappers(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/paul/Library/Caches/pypoetry/virtualenvs/bloom-nofos-ztRUj9S3-py3.11/lib/python3.11/site-packages/django/db/backends/utils.py", line 80, in _execute_with_wrappers
    return executor(sql, params, many, context)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/paul/Library/Caches/pypoetry/virtualenvs/bloom-nofos-ztRUj9S3-py3.11/lib/python3.11/site-packages/django/db/backends/utils.py", line 84, in _execute
    with self.db.wrap_database_errors:
  File "/Users/paul/Library/Caches/pypoetry/virtualenvs/bloom-nofos-ztRUj9S3-py3.11/lib/python3.11/site-packages/django/db/utils.py", line 91, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/Users/paul/Library/Caches/pypoetry/virtualenvs/bloom-nofos-ztRUj9S3-py3.11/lib/python3.11/site-packages/django/db/backends/utils.py", line 89, in _execute
    return self.cursor.execute(sql, params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/paul/Library/Caches/pypoetry/virtualenvs/bloom-nofos-ztRUj9S3-py3.11/lib/python3.11/site-packages/django/db/backends/sqlite3/base.py", line 328, in execute
    return super().execute(query, params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
django.db.utils.OperationalError: no such table: constance_config

System check identified no issues (0 silenced).
......................................................................................................
......................................................................................................
......................................................................................................
......................................................................................................
............................................
----------------------------------------------------------------------
Ran 452 tests in 1.056s

OK
Destroying test database for alias 'default'...

So it does actually run the tests and nothing is failing. But obviously getting a huge error message like that before all the tests run is really annoying.
It seems to be happening by running migrations automatically when creating a disposable test database. It looks to me like the culprit is "0002_migrate_from_old_table.py", line 19 where we look for the table  "constance_config" and don't find it.
To be clear, I am not actually testing constance or mocking any values, it's just that the tests automatically are running these migrations every time I run my test suite.
To get rid of the stack trace, I have added this to my settings.py which prevents running constance migrations during tests.
# Disable migrations for constance in tests because there is a bug in their migration file
if "test" in sys.argv or "test_coverage" in sys.argv:
    MIGRATION_MODULES = {
        "constance": None,
    }

If there is some other way to address this, I am all ears.
System configuration

Django version: 5.0.8
Python version: 3.11.9
Django-Constance version: 4.0.1

I have no idea how this is connected to the django-constance 4.x release.
The code you linked comes from #469. And this is from 3.x release.
@sergei-iurchenko what could have caused this?

It is caused by the mentioned PR, and not tied to release 4.x. Data is migrated from constance_config to constance_constance in migration 0002_migrate_from_old_table. The exception is only logged, and properly handled (as stated - the current state is functional).
The migration logs an error as FROM constance_config fails on fresh databases, as the database name is not stated anymore/not created. I monkey-patched the function with this fully untested snippet:
def _migrate_from_old_table(apps, schema_editor) -> None:
    """
    Copies values from old table.
    On new installations just ignore error that table does not exist.
    """
    connection = schema_editor.connection
    quoted_string = ', '.join([connection.ops.quote_name(item) for item in ['id', 'key', 'value']])
    if 'constance_config' in connection.introspection.table_names():
        try:
            with connection.cursor() as cursor:
                cursor.execute(
                    f'INSERT INTO constance_constance ( {quoted_string} ) SELECT {quoted_string} FROM constance_config',  # noqa: S608
                    [],
                )
                cursor.execute('DROP TABLE constance_config', [])
        except DatabaseError:
            logger.exception('copy data from old constance table to a new one')

    Constance = apps.get_model('constance', 'Constance')
    sequence_sql = connection.ops.sequence_reset_sql(no_style(), [Constance])
    with connection.cursor() as cursor:
        for sql in sequence_sql:
            cursor.execute(sql)

Hello) This is expected correct behaviour. We can do nothing with it to preserve backwards compatibility.
I have to 2 ideas to make it a bit less verbose:

configure logging to suppress error messages from this module
Configure migrate to False in DATABASES https://docs.djangoproject.com/en/5.0/ref/settings/#migrate to create tables in tests by models not by migrations


Fix sample code in README

The sample code was incorrect so fixed it.
Sorry for small correction...

Good catch!
Correction size doesn't matter. :-)


Set display as the default render value in the datatables.js transformer

Fixes #56 Setting the display value as the default column value for the data tables transformer

Coverage remained the same at 97.482% when pulling e85e1ec88eb31a37d56426df626bd27a5694d089 on uni_dim_sort_fix into e924e0f22e06a1a5dd9265966a26ced036d67851 on master.


images

Coverage remained the same at 92.432% when pulling d52c1aab4dbe141fc965caa023896fa442276298 on marqh:11docs into f89fbb26a0d95ea480eb11285c8d6b5343c0f40e on SciTools:master.

Coverage remained the same at 92.432% when pulling a8d879a2c94c20cfb2be226f33ba0f7b0a347dc1 on marqh:11docs into f89fbb26a0d95ea480eb11285c8d6b5343c0f40e on SciTools:master.

Coverage remained the same at 92.432% when pulling a8d879a2c94c20cfb2be226f33ba0f7b0a347dc1 on marqh:11docs into f89fbb26a0d95ea480eb11285c8d6b5343c0f40e on SciTools:master.

Coverage remained the same at 92.432% when pulling a8d879a2c94c20cfb2be226f33ba0f7b0a347dc1 on marqh:11docs into f89fbb26a0d95ea480eb11285c8d6b5343c0f40e on SciTools:master.

Coverage remained the same at 92.432% when pulling 0405d649b219949476982e72428c0eed6101bbb3 on marqh:11docs into f89fbb26a0d95ea480eb11285c8d6b5343c0f40e on SciTools:master.

Coverage remained the same at 92.432% when pulling 2df342643dc0e2e86260c2af06f8000611b6c4a6 on marqh:11docs into f89fbb26a0d95ea480eb11285c8d6b5343c0f40e on SciTools:master.


support having in sql

sql tamplate：
  SELECT [ ALL | DISTINCT ]{ * | projectItem [, projectItem ]* }
  FROM tableExpression
  [ WHERE booleanExpression ]
  [ GROUP BY { groupItem [, groupItem ]* } ]
  [ HAVING booleanExpression ];

test sql:
SELECT Customer,SUM(OrderPrice) FROM XXX
GROUP BY Customer
HAVING SUM(OrderPrice)<2000;

test data:

Customer
OrderPrice

Bush
1000

Carter
1600

Bush
700

Bush
300

Adams
2000

Carter
100

expected result:

Customer
SUM（OrderPrice）

Carter
1700

Hi~ Could I hava a try?

sure, of cause.


rewrite pekko-platform-guide to use pekko artifacts and to remove Akka refs

https://github.com/apache/incubator-pekko-platform-guide
This guide is referenced quite a bit from the main docs associated with this repo.
Edit: the doc refs from this repo have been temporarily removed which means that updating the pekko-platform-guide is not as urgent. See https://github.com/apache/incubator-pekko/commit/658e4bc4b0b9fd0305f435ab02c445a5344783aa
If and when the pekko-platform-guide is updated and published, we could revert that commit and tidy up the links to ensue they point to the right place (where the new pekko-platform-guide docs are published).

@sam-byng I've already removed the pekko doc links to the platform guide docs. See https://github.com/apache/incubator-pekko/commit/658e4bc4b0b9fd0305f435ab02c445a5344783aa
This makes this a nice-to-have as opposed to something that blocks a v1.0.0 release. If we do get the pekko-platform-guide fixed up though, we can easily add back the links.

On doc builds/tests, the current solution seems relatively straight forward:
locally:
make html-author-mode
start target/index.html
In github pipelines: current solution looks fine
https://github.com/apache/incubator-pekko-platform-guide/blob/main/.github/workflows/build-and-test.yml
TODO So next steps here are to do the grunt work of converting the docs themselves.
It looks like test main branch builds were then deployed onto a test VM<EMAIL_ADDRESS>TODO: This will need update to a pekko hosted VM. The exact owner of this etc needs discuss.
On publishing releases:
On akka, the The guide is published via tech hub via a webhook.

TODO: I don't have experience in this so I don't know what the correct process should be here. Possibly some crossover with
https://github.com/apache/incubator-pekko-site
On hosting: See point 2 above. If we decide to host @ pekko.apache.org/ then the discussions on mailing list of [DISCUSS] using a VM to serve the pekko.apache.org web site will be pertinent

Let's not assume that ASF will be able to host a VM. We might be able to but it may not be worth the cost for a non-profit org. If the gustav VM (or possible ASF equivalent) is an important part of someone being able to follow the instructions and run the samples, then we might need to consider whether this guide is a good thing for us.

doc pages can be hosted at https://pekko.apache.org/docs/platform-guide/current - or something similar.

If the gustav VM (or possible ASF equivalent) is an important part of someone being able to follow the instructions and run the samples, then we might need to consider whether this guide is a good thing for us.

AFIACT The use of the Gustav VM in this case is for test purposes. It looks like there's a long-lifetime akka.io website that is kept live and redeployed with the latest main whenever any main pipeline runs. This seems like a non-essential add-on that just lets devs easily view the current state of the docs on a server instead of needing to build locally.
So I propose just remove this since it looks like we won't want to use it.

I'd probably vote https://pekko.apache.org/docs/pekko-platform-guide/current , since the other modules are prefixed with pekko in https://pekko.apache.org/docs/ and the previous URL was @ akka-platform-guide


Distributed secondary index query on hash

Support distributed secondary index query on hash based on the #7 .
Should return the results by the order of the secondary index across the multi partitions in the namespace.

如果没有支持 Secondary index 就不用贴这个标签吧