ModelIndexes not defined using Amplify code gen

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
sudo npm install -g @aws-amplify/cli
If applicable, what version of Node.js are you using?
v17.4.0
Amplify CLI Version
10.0.0
What operating system are you using?
Mac OS Monterey
Amplify Codegen Command
codegen models
Describe the bug
the new codegen models, add modelIndex that was not defined in flutter, i revert back to old amplify 9.2.1, and the error was gone, please fix the new one
Expected behavior
I expected the modelIndex is defined in the flutter
Reproduction steps
amplify codegen models
GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Additional information
No response

@marendra Hi I am not able to reproduce the issue. Could you share your GraphQL schema(amplify/backend/api/${api_name}/schema.graphql) and the feature flag config (amplify/cli.json)?

okay here it is the schema :
type User @model
@auth(
rules:[
{allow:owner,operations:[read,create,delete,update]}
{allow:private, operations:[read]}
]
)
{
id: ID!
user: ID! @index(name:"byUserId",queryField:"queryByUserId")
name: String!
email: String! @index(name:"byEmail",queryField:"queryByEmail")
userName: String! @index(name:"byUserName",queryField:"queryByUserName")
goodDeedsTotal: Int!
followersTotal: Int!
followingTotal: Int!
profileImage: String
backgroundImage: String
userBio: String
created: Float!
birthDate: Float!
latitude: Float!
longitude: Float!
city: String!
}
type Relationship
@model
@auth(
rules:[
{allow:owner, operations:[read,create,delete,update]}
{allow:private,operations:[read]}
]
)
{
id: ID!
followee: String! @primaryKey(sortKeyFields:["followers"])
followeeName: String!
followeeImageUrl: String!
followersName: String!
followersImageUrl: String!
followers: String! @index(name:"searchFollowing", queryField:"queryByFollowing")
}
type Post
@model
@auth(
rules:[
{allow:owner,operations:[read,create,delete,update]}
{allow:private,operations:[read]}
]
)
{
id:ID!
user:ID! @primaryKey(sortKeyFields:["created"])
created: Float!
title: String!
description: String!
imageUrl: String!
}
type Timeline
@model
@auth(
rules:[
{allow:private,provider:userPools operations:[read]}
{allow:private,provider:iam,operations:[read,create,delete,update]}
]
)
{
user:ID! @primaryKey(sortKeyFields:["created"])
created: Float!
title: String!
description: String!
imageUrl: String!
postId:ID! @index(name:"timelineBy",queryField:"queryByPost")
comment:Int!
like:Int!
}
and the cli.json
{
"features": {
"graphqltransformer": {
"addmissingownerfields": true,
"improvepluralization": false,
"validatetypenamereservedwords": true,
"useexperimentalpipelinedtransformer": true,
"enableiterativegsiupdates": true,
"secondarykeyasgsi": true,
"skipoverridemutationinputtypes": true,
"transformerversion": 2,
"suppressschemamigrationprompt": true,
"securityenhancementnotification": false,
"showfieldauthnotification": false,
"usesubusernamefordefaultidentityclaim": true,
"usefieldnameforprimarykeyconnectionfield": false,
"enableautoindexquerynames": false,
"respectprimarykeyattributesonconnectionfield": false,
"shoulddeepmergedirectiveconfigdefaults": false,
"populateownerfieldforstaticgroupauth": false
},
"frontend-ios": {
"enablexcodeintegration": true
},
"auth": {
"enablecaseinsensitivity": true,
"useinclusiveterminology": true,
"breakcirculardependency": true,
"forcealiasattributes": false,
"useenabledmfas": true
},
"codegen": {
"useappsyncmodelgenplugin": true,
"usedocsgeneratorplugin": true,
"usetypesgeneratorplugin": true,
"cleangeneratedmodelsdirectory": true,
"retaincasestyle": true,
"addtimestampfields": true,
"handlelistnullabilitytransparently": true,
"emitauthprovider": true,
"generateindexrules": true,
"enabledartnullsafety": true
},
"appsync": {
"generategraphqlpermissions": true
},
"latestregionsupport": {
"pinpoint": 1,
"translate": 1,
"transcribe": 1,
"rekognition": 1,
"textract": 1,
"comprehend": 1
},
"project": {
"overrides": true
}
},
"debug": {
"shareProjectConfig": true
}
}

also here is the error message in the flutter models/user.dart

@marendra Just consult with the amplify flutter team. The minimum version supporting the index is<EMAIL_ADDRESS>Could you update the amplify flutter library and check if the error still exists?

okay it works after change all to 0.6.7, currently im using the version 1.0.0-next.0+1, after change it all to 0.6.7 then it worked

thanks,closing the issue


peerDependency range changing unnecessarily

Affected Packages
@changesets/cli
Problem
When updating dependent package range is changed unexpectedly
So for example, I have:
  "peerDependencies": {
    "@package/common": "workspace:>= 1.7.0 < 2"
  },

But it get's changed to
  "peerDependencies": {
    "@package/common": "workspace:>=1.9.0"
  },

When @package/common is updated.
How do I stop this from happening since I still want the dependency range to stay the way it was?

I'm not sure if it's for the same reason, but I have a similar issue happening with a local devDependency. It seems that this line in the apply-release-plan. For example, it returns false when checking if a version such as 4.3.2 satisfies a range like workspace:^4.0.0, when it should be actually true.


increased output token limits for 3.5 sonnet

Requirements:

Increase output token limits for 3.5 sonnet from 4000 to 8000
Context: https://sourcegraph.slack.com/archives/C05SZB829D0/p1721069882054949

Test plan
manually test

Left some comments inline.
This change alone will not work since the header "anthropic-beta": "max-tokens-3-5-sonnet-2024-07-15"  needs to be included on the server side when server side make the API requests to Anthropic, so might need to get cody prime team to make that change first. The Cody prime team will also need to update the current output limit to 8k or this change will trigger the abuse detection IIRC?

Thanks @abeatrix . This is where my knowledge ends, and although I used Cody Web to try to move me forward I am stuck. CC: @sourcegraph/cody-prime to help!


When first added opens screen asking for data with no field discriptions

Home Assistant 2020.12.2

Is this when you install through HACS? Where do you see this screen?
If it’s in the add integrations page, installation is performed through the configuration.yaml file.

Is this when you install through HACS? Where do you see this screen?
If it’s in the add integrations page, installation is performed through the configuration.yaml file.

Is this when you install through HACS? Where do you see this screen?
If it’s in the add integrations page, installation is performed through the configuration.yaml file.

Apologises I should have provided more information.
It was after a manual install.  I copied files to components. The window popped up when adding the integration within HA.
I simply entered junk text in the pop up and it worked fine.
Edited configuration.yaml with my api etc.

Is this when you install through HACS? Where do you see this screen?
If it’s in the add integrations page, installation is performed through the configuration.yaml file.

Apologises I should have provided more information.
It was after a manual install.  I copied files to components. The window popped up when adding the integration within HA.
I simply entered junk text in the pop up and it worked fine.
Edited configuration.yaml with my api etc.

No problem, I think I can see why this is happening, will have a look to try and replicate and stop it.

No problem, I think I can see why this is happening, will have a look to try and replicate and stop it.

Just having a look, did you go to "Add integration" when you got this issue? Or did it pop up before that point?

Just having a look, did you go to "Add integration" when you got this issue? Or did it pop up before that point?

Yes correct - At point of Add integration.  (I also tried 'Add integration' this after installing via HACS and same happened). I guess I was more puzzled by the empty field names  - Have in mind I set up configuration.yaml before installing my HACS

Yes correct - At point of Add integration.  (I also tried 'Add integration' this after installing via HACS and same happened). I guess I was more puzzled by the empty field names  - Have in mind I set up configuration.yaml before installing my HACS

That makes sense. It's because I hadn't setup the ability to add the integration through the GUI. There isn't any need to go to Add Integrations as you have already configured the integration through configuration.yml.

That makes sense. It's because I hadn't setup the ability to add the integration through the GUI. There isn't any need to go to Add Integrations as you have already configured the integration through configuration.yml.

I was also taken in by this - it seemed "obvious" to me that the dialogue was asking for MPAN, MeterID and API_key ...


Primary Key for Migration Lock Table

In reference to #2534, the single line change has been run with tests against DB='mysql' npm test.
Summary
  total:     84
  passing:   84
  duration:  1.6s

Full result
testIssue2534.txt

Please note that this PR does not result in a documentation change, so does not reference a documentation branch PR.

Updated PR to include creation of an increments 'id' column as primary key as suggested.

Looks like there was a glitch with the Travis CI build (Oracle download error - they have very poor open source hygiene in my experience).  Closing an reopening PR to trigger another build.

Reopening to trigger new build.

Looks great, thanks!

Thanks - happy to be able to contribute.


Please publish latest version on registry

Please publish v1.1.0 on https://registry.terraform.io/modules/zoitech/account/aws/latest

@Geartrixy Just running though my routine updates and circling back to this.


Adding request to the curl and http validation step

Adding request to the curl and http validation step to improve how the output is displayed.

@felderi what does this change add? I tested it everywhere, the output seems to be the same with or without /request. I guess if you already have plugins enabled on that route, it might get messy? That shouldn't be the case with the getting started guide, since they're ostensibly getting started on Kong and have nothing else enabled yet.
If you think this is necessary after all - can you also edit the browser request path in line 140?

@felderi what does this change add? I tested it everywhere, the output seems to be the same with or without /request. I guess if you already have plugins enabled on that route, it might get messy? That shouldn't be the case with the getting started guide, since they're ostensibly getting started on Kong and have nothing else enabled yet.
If you think this is necessary after all - can you also edit the browser request path in line 140?

Without the /request, the response is the content of the website mockbin.org which is messy.  When you add the /request that brings back a formatted json response.  We can go through it together if you like.

Without the /request, the response is the content of the website mockbin.org which is messy. When you add the /request that brings back a formatted json response. We can go through it together if you like.

Interesting. I don't have that happen with the GET request for validation, I only see that formatting when the route is created. Environment differences?


bson.errors.invalidDocument: key * must not contains '.'

When using a real mongodb database I can include a entry like a below, but in mongomock it not possible.
{"object.name": value}
When I add this in real database it works, but in mongomock doesn`t.

Just encountered the same problem..  Any update on this issue?

Thanks for reporting this. Handling key with dots is not supported by mongomock yet. PRs are welcomed though.


Heat pumps heats up houses over upper temperature boundary.

Target temperature: 20
Lower limit: 18
Upperl limit: 22

As seen in the picture the house is heated up to almost 23°C when simulating with a cylindrical storage at the house without an EnergyManagementAgent.

When simulating without a thermal storage the house is overheated less.
It can be discussed wether to stop at 20°C or 22°C.

This seems to happen only for heatpumps that are not under the controll of an EmAgent

Result of further investigation:

Activation of Hp (not under control of EmAgent) is working fine when reaching the lower temperature of it's thermal house.
Running Hp get stopped to the next full hour regardless the target temperature has not been reached, though. Most probably this is caused by new weather data.

Side result: CylindricalStorage will not be used since without Em there is no surplus energy.
ThermalHouse Results

Hp Result

Result of further investigation:
Heatpump not under control of EmAgent

When reaching the lower temp boundary of thermal house the Hp is start working -> 2019-01-18 08:00 (check why activated at the next full hour)
Hp heat up house till upper boundary -> 2019-01-18 12:00 (to be discussed)
Hp heat storage afterwards -> 2019-01-18 12:00 (good)
House got overheated again -> 2019-01-18 14:00 (to be checked why)
Hp runs longer than expected -> 2019-01-18 15:00 together with double and wrong storage result entries.

ThermalHouse Result

Hp Result

Storage Result

Storage will be used to cover heat demand without starting Hp -> 2019-01-19 07:00. (good)
When storage is empty the heat pump will not be started immediatly to refill the storage. It only starts, when the thermal house temp drops. (Should be improved)
ThermalStorageResults have multiple entries (2019-01-19 18:00, 2020-01-20 02:00), while the hour following to the double entries is missing.
qDot result still positive when storage is already full (2019-01-20 04:00), no exact entry when storage reaches full charge.

CylindricalStorage Result

Hp Result

ThermalHouseResult


Enable most feature tags for CI builds, cleanup

Enabled most feature tags for CI builds:

some flags would require installing additional C development libraries in github workflow (to be installed via apt prior to Go build)
some flags trigger Go build problems (to be investigated & fixed)

Fixed typo b_bctypt -> b_bcrypt
Cleanup bad Go module practice via go-get-*.bash files & all references as the dependancies were put into proper go.mod & go.sum files in prematurely merged #52

Thank you! Problematic modules can be "archived" for now, many of them are just P-O-C and are not needed right now by anyone. They would be reinstated when / if there is interest.


MWPW-157999 - [Table] Add-on content

As a Milo Engineer, I want to add functionality to Milo table block so I can add design features from this design.

New features Include:

Rendering additional 'add-on' content in the table headers:
The table header decorator function requires that the content be written in a specific order. This PR introduces an additional method of inserting content into the header columns. Utilizing additional rows that are titled with an ADDON key. To use, add the position, order, and optional tag name separated by underscores to the row titles in these additional header rows and the table block will handle the rest. Example: ADDON_BODY_AFTER_PROMO.

Rendering label/badge styles in the table body:
To provide support for this design we needed to be able to create a custom element style in the table body. We did this by authoring the desired content with the quote style in word. When using add-on content the table block will style any blockquote content in the row titles to be styled like a label/badge.

Resolves: MWPW-157999
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/sarchibeque/table/add-on?martech=off

After: https://sartxi-mwpw-157999-table-addon--milo--adobecom.hlx.page/drafts/sarchibeque/table/add-on?martech=off

Before: https://main--milo--adobecom.hlx.page/drafts/sarchibeque/table/add-on-greentext?martech=off

After: https://sartxi-mwpw-157999-table-addon--milo--adobecom.hlx.page/drafts/sarchibeque/table/add-on-greentext?martech=off

Looking good. Few super minor design things to touch on if you think there worth it.

The addon-label icon can get pretty close to the text, maybe give it a little margin-start.
The table-title-text is italic, though it is authored w/ an<em>, so maybe its just a doc mistake.

Should both this content use the same authoring style (quote or code, etc) since it looks similar? maybe it's diff enough, just a though.

Looking good. Few super minor design things to touch on if you think there worth it.

The addon-label icon can get pretty close to the text, maybe give it a little margin-start.
The table-title-text is italic, though it is authored w/ an<em>, so maybe its just a doc mistake.

 Should both this content use the same authoring style (quote or code, etc) since it looks similar? maybe it's diff enough, just a though. Oop, the font color too on these looks a lil off.

Okay I have addressed your findings, thanks!

test loc workflow and add any exceptions in the htmlFlow for cell keys ADDON etc

Thanks for the reminders, I will make sure get them done!

Sub-Task for localization workflow updates
Sub-Task for authoring documentation


PSReadline again

Is there any news on PSReadline/*nix issue?
I'm looking for some work around that allows to use Polaris and keep PSReadline. I guess you mention this can be fixed by replacing Register-ObjectEvent whith a while loop. Is it an easy change? If so, can you show any sample code?
Btw, I noticed that this issue disappears if starting Polaris from VSCode Powershell Integrated console

Let's keep the discussion for this in #136 just so any workarounds we discover or are mentioned can be found in the single master thread. I'll repost your question over on that thread and close this one out.


Spack package debug variant doesn't add debug symbols

The combination of adding flags to CXXFLAGS in your spack package and your makefile not adding DEBUG_FLAGS to CXXFLAGS if its not empty (line 200 of your base Makefile), you end up with no "-g -O0" in your debug variant.
The following is one example of a fix is to change the following in your spack package from:
        if cxxflags:
            cxxflags = [(xcompiler + flag) for flag in cxxflags]

to:
        if cxxflags:
            cxxflags = [(xcompiler + flag) for flag in cxxflags]
            if '+debug' in spec:
                cxxflags.extend(['-g', '-O0'])

If this fix is acceptable I can push the fix to Spack repo in a PR I am doing soon for Axom and Conduit.

This is the intended behavior -- if you set the cxxflags in Spack they replace the default values in MFEM, regardless of the debug variant. If using +debug, just set the cxxflags to the desired combination of flags.
Note that +debug does more than just switch the flags. For example, it is sometimes useful to enable +debug and still use -O3 optimization -- which will not be possible with your suggestion.

I just saw the bug that was causing it in my version of spack was fixed.  The C++11 flag was always being added in my version and can update to the newer one that doesn't.
This is more of a philosophical argument at this point and if you expect your users to set the optimization/debug flags in their global flags that's fine, I can work around it.  The problem is I am needing to add a default flag ("--gcc-toolchain=/newer/gcc") across the board and that causes both the debug and opt flags to never be set.
You can feel free to close this issue.

After thinking about this more and consulting @becker33 on the Spack team, I may have snippet of code that should work for both of our needs.  Basically, if cxxflags is set by the user, it tries to find debug or opt flags and if they are missing adds them.  This requires a fix on Spack's side and injecting this (tentative) code after line 368 in your current mfem/package.py:
`
opt_flag_found = any(f in self.compiler.opt_flags for f in cxxflags)
debug_flag_found = any(f in self.compiler.debug_flags for f in cxxflags)
        if '+debug' in spec:
            if debug_flag_found:
                cxxflags.append('-g')
            if opt_flag_found:
                cxxflags.append('-O0')
        else:
            if opt_flag_found:
                cxxflags.append('-O2')

`
I would really like to find a solution that both of us are happy with so I can stop keeping a copy of your package in my project's repositories (Axom and Serac).  This just causes us to become out of date with your current package as well as pushes our shared solutions to the rest of the Spack community.

Absolutely, we want to avoid custom copies too and other users are probably running into the same issues as you.
Will it help if we have a short telecon.

Let me test out the branch @becker33 just created and get back to you.  It actually requires two branches and one should be approved soon.

https://github.com/spack/spack/pull/17121
Sorry this was a very long running set of fixes to get to this point (thanks @becker33)!  I have created a PR in the spack repo for this fix


Add the identity map client docs for python sdk

I discussed this recently with Gen in context of EUID docs. The Java SDK doc doesn't mention the functionality of mapping DII to raw UID in the "Functionality" section. Gen added it to EUID docs https://euid.eu/docs/sdks/sdk-ref-java#functionality May be good to include that too

Yeah, that is a good point, I added it


nixos-container: fix show-ip command to support IPv6 addresses

Motivation for this change
I use nixos containers with IPv6 addresses only. nixos-container get-ip <container> prints /run/current-system/sw/bin/nixos-container: cannot get IP address
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Do you use imperative contains (i.e. nixos-container create) or declarative containers (containers.foobar in <nixos-config>)?
Also IIRC LOCAL_ADDRESS6 is used (by the declarative module at least) to assign an IPv6 address to a container, so this change seems confusing to me.

Oh, that might be me using the configuration wrong.
I use declarative containers like this:
  containers.foobar = {
    autoStart = true;
    privateNetwork = true;
    hostAddress = ipv6Prefix + "::1";
    localAddress = ipv6Prefix + "::2";
    config = { config, pkgs, ... }:
    …

and boot.kernel.sysctl."net.ipv6.conf.all.forwarding" = 1; in the host.
It has been working just fine so far except nixos-container show-ip foobar.

First of all, any reason you're not using hostAddress6 and localAddress6?
In that case you could look for LOCAL_ADDRESS and LOCAL_ADDRESS6 in nixos-container.pl.

Not at all, I didn't know localAddress6 was a thing.
I'll try to adjust the script accordingly.

@Ma27 Ready for review

@Ma27 can we merge?


Fix tf conversation

Currently eagleye does not take roll and pitch angles into account when converting position with tf.
This PR implements a tf conversion that takes roll/pitch into account.
Concerns
Current eagleye estimated roll angle is noisy

This PR is temporarily closed because the current eagleye estimated roll angle is noisy and greatly degrades the accuracy of trajectory estimation.


Add the header itself as a release asset

Hi, and thanks for your work in this and the other -lite repos.
I think it would be convenient to have the header file made directly available as a build asset, so those who only want the .hpp can get it without downloading and unzipping the whole repo archive.
A random example of a C++ header-only library on GitHub that does this is rang:

This suggestion applies to all the other -lite libraries too, of course.

Thanks for bringing up this this need.

Now done for expected, optional, value_ptr, variant.

Brilliant! I'll close this issue.


[BUG] Can't exit fullscreen when launching Roblox with Bloxstrap

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
I can't exit fullscreen when launching Roblox with Bloxstrap, I could exit fullscreen it if i launch Roblox without Bloxstrap, btw i'm using nvidia drivers

nvm its just cuz im using vulkan


Overloaded indexing doesn't support mutation

Now that we can use __index__ for user types, it seems natural to also support indexed assignments as well.
foo ::= struct {
  x: i64
}

__index__ ::= (self: *foo, idx: i64) => self.x

f := foo.{ x = 3 }
f[0]
f[0] = 1

Results in the following error:
Error in repro.ic:
Assigning to a non-reference expression:

  10 | f[0] = 1

[139632588666696 compiler/emit/index.cc:112 EmitRef] Unreachable code-path.
(struct.57625264)
*** SIGABRT received at time=1639544198 on cpu 0 ***
PC: @     0x7efebee2a808  (unknown)  pthread_kill
    @          0x1923ef0         64  absl::WriteFailureInfo()
    @          0x1923bd4        224  absl::AbslFailureSignalHandler()
    @     0x7efebedd6520  (unknown)  (unknown)
Aborted (core dumped)

I also tried changing __index__ to return a pointer to i64, but that resulted in one additional error along with the results above:
Error in repro.ic:
Cannot assign a value of type `integer` to a reference of type `*(i64)`:

  10 | f[0] = 1

Agreed. The underlying issue is that the language doesn't have references, only pointers.

I wonder if it's as simple as this:
Assignments take a left-hand side of type *T and right-hand side of type T.
Because we already have implicit conversions from T to *T, for primitive types, something like:
n: i64
n = 3

Would be correctly interpreted as implicitly casting the left-hand side to the address of n, and assigning 3 into that address`.
When I describe this to myself in terms of pointers, it feels very strange. But rethinking this in terms of references (here I'm using the C++ terminology of "pointers" and "references") this seems to make a lot more sense. Effectively in C++ a user-defined operator= is a function accepting a pointer/reference to T and a T, and we rely on the implicit conversion from T to reference-to-T (when the value category matches).
I'm not sure if there's something about the syntax we're using for pointers that makes this feel weirder, or it's just my bias coming from C++. I'm also not entirely sure this approach is sound, though I haven't found any holes in it yet.
Thoughts?

What happens if the lhs is const or doesn't have a reference type that can be assigned to? (What are examples of such a thing?)
What happens if you try to take a reference of an expression?
Eg
2+2 = 5

The implicit cast to reference works the same as C++: the compiler tracks the value category and the cast is only allowed for lvalues.
There is still an issue where we want references to implicitly dereference so that c := s[0] copies the strings first character. Or maybe that doesn't always happen?
C++ always dereferences. I believe Rust doesn't. Rusts approach is certainly more confusing at first glance (the rules are objectively more complex) but the borrow checker alleviates this. So maybe in c := s[0] it is a reference to a character, but the borrow checker will let you know if you accidentally assign through c.
Also strange: pointers/references can be rebound with this approach via
&p = new_value

Ideally we'd have a way to trigger user-defined logic when index-assigning. The example that comes to mind is a sparse matrix type, where you might need to do some non-trivial work on assignment:
__index__ ::= (self: *sparse_array, idx: i64) -> f64 {
  if (self'in_sparsity_pattern(idx)) {
    return self'explicit_value(idx)
  }
  return 0
}

__index_assign__ ::= (self: *sparse_array, idx: i64, val: f64) -> () {
  if (self'in_sparsity_pattern(idx)) {
    self'set_explicit_value(idx, val)
  } else {
    self'update_sparsity(idx, val)
  }
}


kB convert with 1024

Conversion to human size should be with a factor of 1024 and not 1000.

@betizad - just following up with my previous comment.  If you have a second, can you please respond?

Thanks for the clarification. Indeed, I just realized that.
manufacturers often use the term "kilobyte" to mean 1024 bytes (a kibibyte).
The helper function "humanize_bytes_binary" would be really appreciated.

@betizad - cool, would you like to submit a PR to add the humanize_bytes_binary.  We'd love to have your contribution.  If you don't want to add this that's alright too.  I'd also be happy to add it.  Thanks again for the valuable suggestion.

Sounds good. I'll send a PR.

@MrPowers The changes are pushed.

@betizad the GitHub actions for black and flake8 are failing, could you make sure to run the pre-commits on your side before pushing?

@betizad - thanks for making the contribution.


Make the servlet API a runtime provided dependency when using JPMS.

When building a custom runtime for a Wicket application a servlet container will have to be required which will pull in the dependency. Symmetrical with how it works when using Maven to build a WAR file that is deployed to a stand-alone application server.

I'm curious about how is that supposed to work? There are many code places requiring HttpServletRequest, doesn't this break when javax.servlet isn't available?

The module is still available at compile time. This is just bringing it to parity with how regular WAR deployment works. When building a WAR file the servlet specification is not included in the WAR file but instead provided by the application server (servlet container, like Tomcat). Similarly if you're building a trimmed down runtime image of a Wicket application an embedded application server will have to be required which would in turn pull in the servlet specification and implementation.

@martin-g have you had the chance to look at this proposed PR? Looks good to me.


unpinned variable

(I still need to verify this bug as this comes from static analysis)
guerrilla.go:132:30
Using a reference for the variable on range scope sc
server, err := newServer(&sc, g.backend(), g.mainlog())
So the value of  "&sc" will be always the same as it unpinned, right?
Fix would be to pin with  sc := sc
// Instantiate servers
func (g *guerrilla) makeServers() error {
	g.mainlog().Debug("making servers")
	var errs Errors
	for _, sc := range g.Config.Servers {
		if _, ok := g.servers[sc.ListenInterface]; ok {
			// server already instantiated
			continue
		}
		if err := sc.Validate(); err != nil {
			g.mainlog().WithError(errs).Errorf("Failed to create server [%s]", sc.ListenInterface)
			errs = append(errs, err)
			continue
		} else {
			server, err := newServer(&sc, g.backend(), g.mainlog())
			if err != nil {
				g.mainlog().WithError(err).Errorf("Failed to create server [%s]", sc.ListenInterface)
				errs = append(errs, err)
			}
			if server != nil {
				g.servers[sc.ListenInterface] = server
				server.setAllowedHosts(g.Config.AllowedHosts)
			}
		}
	}
	if len(g.servers) == 0 {
		errs = append(errs, errors.New("there are no servers that can start, please check your config"))
	}
	if len(errs) == 0 {
		return nil
	}
	return errs
}

Thanks, indeed - the pointer would get clobbered. Should have been using the index instead of value when getting the reference. So change the range so that it also gets the index i, then change
newServer(&sc, g.backend(), g.mainlog())
to
newServer(&g.Config.Servers[k], g.backend(), g.mainlog())
Not sure why this hasn't been caught earlier, but nice catch! Do you want to submit the PR?

Yes, I will try to fix all the issues I have submitted. Just wanted to discuss with you beforehand.

Hi @tegk eagerly awaiting your PR :-)
Since this issue contains a bug, it has a more significant priority, and would like see this fixed this soon. Thanks.


Remove Mutation Flow

Purpose
Changes "click here" link on Linode Mutation flow from opening up the mutation drawer to redirecting the user to CF Manager in a new tab
Also removes Opt Out of Mutation feature altogether, as per review
To Test
Login to dev-test-003. There should be one more Linode that has a pending mutation available

Coverage increased (+0.01%) to 26.278% when pulling 4bc946d10620d58d3e19db733c3561e5621d883a on martinmckenna:remove-mutation into 7087cbf017e52d21187e56920eb60afa7d975303 on linode:release-0.33.1.


Operator Configuration - support all sorts of inputs

Internal conf package
Using Viper support setting the configuration via files, env and parameters
main.go
Load all the configuration using the new package
Remove previously defined flags
Get the configuration values from the conf pacakge
Manager Deployment
Attach the configuration configMap.
Set the mount point of the configuration to etc
Do not pass any arguments as the values will be taken from the file
Pass the operator namespace using the environment variable resolved from the deployment spec
Vendor
Add spf13/viper and its dependencies
Misc
Ignore vscode directory

Why is there a need to load configuration from any other source than env variables? It looks way more complicated than simple envconfig usage.

The operator SDK already creates the configMap and I think that a configuration file wrapped in a ConfigMap is cleaner than environment variables. Plus this implementation does not break other configuration options (env or flags)

That config map can be used as env source for the container - why not use it in a simple way, like in the Flotta Operator? I don't think there is a need for additional configuration abstraction (indirection) in the shape of a file wrapped in a ConfigMap.

Why is there a need to load configuration from any other source than env variables? It looks way more complicated than simple envconfig usage.

The operator SDK already creates the configMap and I think that a configuration file wrapped in a ConfigMap is cleaner than environment variables. Plus this implementation does not break other configuration options (env or flags)

That config map can be used as env source for the container - why not use it in a simple way, like in the Flotta Operator? I don't think there is a need for additional configuration abstraction (indirection) in the shape of a file wrapped in a ConfigMap.

But the mechanism for the ConfigMap is already in place, I'm not inventing anything. Plus, that configMap is a single entry one designed to be a file, not parsed into envs

Please have a look at Flotta operator what I have in mind instead:

config map generator creating default CM with env variables for the operator
use of the map
config model
config model loading - one line instead of byzantine (in comparison) use of Viper.

Please have a look at Flotta operator what I have in mind instead:

config map generator creating default CM with env variables for the operator
use of the map
config model
config model loading - one line instead of byzantine (in comparison) use of Viper.

I see your point. One thing though, I think I would still use a separate package for the configuration so that it is accessible to other parts of the code (and not through the main package). WDYT?
BTW in flotta-operator, do you use the controller_manager_config.yaml file?

I see your point. One thing though, I think I would still use a separate package for the configuration so that it is accessible to other parts of the code (and not through the main package). WDYT?

Strong yes for that part - I've done something similar for the edgeapi.

BTW in flotta-operator, do you use the controller_manager_config.yaml file?

That's a very good question - it seems to be unused and probably should be removed.

@jakub-dzon Done


Missing param for 'ResetRoleInheritance'

Doco doesn't mention the flag for 'reset role inheritence' - with the ability to 'delete inherited permissions'

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: dbab5a2d-0c37-bb2c-f2d9-989bc2a59d05
Version Independent ID: 2f512ab6-be40-1fb2-9f8a-d1bc692537ce
Content: Set-PnPList (sharepoint-pnp)
Content Source: sharepoint/sharepoint-ps/sharepoint-pnp/Set-PnPList.md
GitHub Login: @techwriter40
Microsoft Alias: kirks

@officedocsbot assign @yogkumgit

Hello @kachihro
I need to kindly redirect you to another resource, PnP cmdlets issues should be opened here please first do a search to see if it already exists.
PnP cmdlets are community-driven, that means that anyone can propose changes to them thru a pull request into the source code
PnP cmdlets documentation is directly reflected from CmdLet's source code documentation, if you decide to propose adding the documentation for that parameter please read and follow the PnP guidelines.
Thank you

@get-itips  Thank you very much for the contribution and sharing this explanation. @kachihro
Hope this comment is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

Thanks - but I didn't actually submit an issue - I'd added a page comment here ;
https://docs.microsoft.com/en-us/powershell/module/sharepoint-pnp/set-pnplist
So - you can reopen, or redirect wherever.   I was just adding a comment.

@kachihro Certainly, and your comment is welcome, I said issue because when you add a comment at the bottom of an article, you are actually opening an issue in the GitHub issue system. Thank you!


[chore] Fix request headers

Notes for Reviewers
This PR fixes the request headers for requesting MesheryAPI and MesheryCloudAPI by setting the missing provider-token and provider key in cookies
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

After merging this PR we still need to fix the endpoint for triggering the workflow. Currently its seems bogus as its returning 404.
I don't remember the endpoint so if any of maintainer can please update the endpoint then we should be good 👍

Updates:

Added support to trigger snapshot workflow in meshery/helm-kanvas-snapshot
Added missing kanvas snapshot workflow
Removed required email field

This is how it would work:

@leecalcote @jerensl
Though the snapshot service it self is failing for past 2 days, once its fixed then we should be good.
https://github.com/meshery/meshery/actions/workflows/kanvas.yml
https://github.com/meshery/meshery.io/actions/workflows/kanvas.yml

Can someone with privileges add the missing GitHub access token that can trigger workflow? It would fix the failing check. Then we can merge and release
https://github.com/meshery/helm-kanvas-snapshot/actions/runs/12338342779/job/34433184955?pr=71

Can someone with privileges add the missing GitHub access token that can trigger workflow? It would fix the failing check. Then we can merge and release
https://github.com/meshery/helm-kanvas-snapshot/actions/runs/12338342779/job/34433184955?pr=71

GH_ACCESS_TOKEN is already being set as repository secret

@theBeginner86 Is this a temporary fix for the unmerged PR in the meshery cloud that already handles triggering of the workflow?


Power Collections - Specialized Hashmaps API Proposal

API Proposal for #2406
Rationale
The BCL Dictionary<,> class serves fairly well as a "safe default" hashmap implementation. But that flexibility and resistance to misuse comes at a substantial cost to best case performance, and it is further weighed down by the need to maintain backwards compatibility with decisions that, after 15+ years of hindsight, seem rather less than ideal. The performance ceiling for even naive hash map implementations is easily several times faster than Dictionary<K,V>; scenarios that demand high performance need better performing collections and will gladly tolerate fewer guard rails to get them.
Goals

Performance. All of the performance. "Roll your own" should not be an easy performance win, with benefits too substantial too ignore.

No mandatory virtual dispatch on hot paths. Put the code size vs. abstraction trade-off  in the hands of consumers.
No modulo prime bucket mapping (simple bitmasks & power of 2 sizes)
No separate hash code & key storage & comparison by default - (int, TKey) tuple keys can be used in the rare scenarios where it does make sense
Skip the versioning. Trust me, I know what I'm doing... or I'm at least willing to accept unpredictable runtime behavior if I screw up in exchange for that extra bit of performance.
Maximize cache friendliness for large maps (minimal per-entry overhead costs - use open addressing scheme that gracefully handles high capacity factor, e.g. robin hood or hopscotch)

Pay for play AOT compilation - minimal API surface area, minimal supporting types; leave functionality to extension methods where possible.
Support specialized hash maps (e.g. incrementers) without needing to replicate hash algorithm implementations.
Safe external API - keep dangerous & unsafe behaviors protected so that library builders (such as myself) can safely & confidently hand these out to consumers without copying.

Proposed API
Base hashmap class. No public members - support a quality hashmap implementation to serve as the base for a wide variety of specialized collections, both power collections library provided and consumer implemented.
public abstract class BaseDictionary<TKey, TValue>
{
  protected BaseDictionary(int initialSize);

  protected interface IKeyHandler<TKey>
  {
    uint MapKeyToBucket(TKey key);
    bool KeysEqual(TKey lhs, TKey rhs);
  }

  protected struct EquatableKeyHandler : IKeyHandler<TKey>
  {
    public uint MapKeyToBucket(TKey key) => (uint)EqualityComparer<TKey>.Default.GetHashCode(key);
    public bool KeysEqual(TKey lhs, TKey rhs) => EqualityComparer<TKey>.Default.Equals(lhs, rhs);
  }
  protected uint EntryCount { get; private set; }

  protected void Initialize(int powerOf2Size);
  protected void RemoveAll();
  protected bool RemoveEntry<THandler>(TKey key, THandler keyHandler) where THandler : IKeyHandler<TKey>;
  protected ref TValue FindEntry<THandler>(TKey key, bool insertIfNotFound, out bool found, THandler keyHandler) where THandler : IKeyHandler<TKey>;
  private void Resize<THandler>(THandler keyHandler) where THandler : IKeyHandler<TKey>;

  public struct Enumerator : IEnumerator<KeyValuePair<TKey, TValue>> { /*Enumerator Stuff */ }
}

Basic Dictionary implementation, except faster & with single lookup add & update support
public class BoringButFastDictionary<TKey, TValue> : BaseDictionary<TKey, TValue>, IDictionary<TKey, TValue> where TKey : IEquatable<TKey>
{
  public TValue AddOrUpdate(TKey key, Func<TKey, TValue> addValueFactory, Func<TKey, TValue, TValue> updateValueFactory);

  public TValue GetOrAdd(TKey key, Func<TKey, TValue> valueFactory);

  //Everything IDictionary<,> demands
}

Counting Dictionary
public class CountingDictionary<TKey> : BaseDictionary<TKey, int>, IEnumerable<KeyValuePair<TKey, int>> where TKey : IEquatable<TKey>
{
  public Enumerator GetEnumerator();
  public int Increment(TKey key);
  public int Decrement(TKey key);

  public int this[TKey key] { get; set; }
}

Indexed Set - support indexed lookup, where the key is derived from the value (e.g. msbuild PropertyDictionary)
\\TODO

Super Basic, Broken Prototype
This prototype is provided as a comprehension aid & basis for discussion; it is entirely untested and has rather substantial deficiencies.
https://gist.github.com/Zhentar/eac2d9078860c29c58575e04fbe1deca
Open Questions

Initialization & Resizing - I know there are plenty of very good reasons to want something other than the basic "allocate one contiguous array" backing scheme. But I haven't given this much consideration because it isn't important in my personal use cases. How can alternative strategies be supported without overhead on lookups? Is there a better scheme to use by default?
Is it a good idea to have a fully functional IDictionary<K,V> implementation? or should it be read only given less robust handling of poor implementations?

Make it a constructor parameter?

Value indexes. I haven't thought through any implementation yet, but PropertyDictionary reminded me that it's not an uncommon scenario (I've done it plenty of times myself), and would be easily supported
Is there a good way to keep a safe, clean public API for GetOrAdd & AddOrUpdate without delegate allocations?

Thanks for kicking this off.
API

No mandatory virtual dispatch on hot paths. Put the code size vs. abstraction trade-off in the hands of consumers.

I assume you mean on the comparers, because you made FindEntry and RemoveEntry virtual. For the comparer, you are calling through an interface. I am not sure of the relative cost of that vs. a delegate or virtual call (@jkotas?). If it's non trivial one could imagine a dictionary specialized specifically for, say, StringComparer.OrdinalIgnoreCase. As an alternative to inheritance, partial classes might be sufficient to create specializations like this. There is always a tradeoff between customization and performance, I suggest strong bias here to performance rather than trying to find another balance different to Dictionary<K,V>.
As a general point ideally the API would be as similar as possible to the existing Dictionary<K, V> to make it easier for existing code to use. There should be good reason for divergence.
Performance
All that matters for performance is actual measurements (especially against Dictionary<K,V>) but it would be nice to pick a reasonable reference implementatoin to measure, based on data others have gathered already. There is plenty. I found this an interesting performance overview of collision, growth, and layout strategies: https://bigdata.uni-saarland.de/publications/p249-richter.pdf
(It also compares hash functions, which are fixed for our work here)
For collision - chaining, linear probing, quadratic probing, robin hood, cuckoo etc - it finds robin hood fastest or near fastest for lookup heavy work, and competitive for read/write. Footprint seems comparable for all probing strategies. For robin hood, it discarded various probing heuristics in favor of linear search, and rehash instead of tombstones (I am not sure whether this is the same as this).
For growth - if I read correctly (Fig 5), performance at 90% capacity does not seem much worse than 70% so perhaps resizing can be delayed a little longer than the 72% that System.Collections.Hashtable used.
For layout, your prototype has key and value together (AoS). Since we are getting a chance to modernize,, it would be nice to consider possible SIMD friendliness - the paper suggests SoA is more friendly because the keys are already packed. Without SIMD, they are similar except at low load factors where AoS wins.

No modulo prime bucket mapping (simple bitmasks & power of 2 sizes)

Any particular reason? Some collision resolution strategies require a primal size, right? And growth strategy should be based on measurements.
We should probably use ArrayPool - although if the arrays are of structs specific to this implementation, it would need its own pool.
Depending on the real world importance of excess space and the cost of expansion, it may be useful to be able to specify initial capacity, possibly to offer EnsureCapacity() and TrimExcess() (as Dictionary<K,V> now has). Hashtable also accepts a load factor hint - that seems a bit like "throwing up hands".
Performance scenarios
It would be helpful to create (or reuse) some benchmark scenarios, bearing in mind -

How important is lookup vs. insertion vs remove? My guess for typical .NET use is that although there are plenty of static lookup tables, dictionaries are very often dynamic
How often are lookups successful?
What are the ranges of typical entry counts/sizes of real world dictionaries?
What are the typical specializations of Dictionary<K,V> and with what comparers? It may be interesting to optimize for the most common K's - I would guess string is top.

For the last two we can probably dig up some statistical data based on a corpus we have.
@benaadams @jkotas @vancem for thoughts.

Is it a good idea to have a fully functional IDictionary<K,V> implementation? or should it be read only given less robust handling of poor implementations?

I think the most common case by far is to use a dictionary internally rather than pass it to 3rd party API. Also as a datapoint in a quick grep of corefx\src*\ref**cs, not many API accept IDictionary<K,V>, especially if you exclude those that are just initializing themselves. So perhaps you can ignore the interface for now.

Value indexes. I haven't thought through any implementation yet, but PropertyDictionary reminded me that it's not an uncommon scenario (I've done it plenty of times myself), and would be easily supported

Do you mean a bidirectional map? That seems something to worry about later. (If by PropertyDictionary you mean this, then it is not a bidirectional map, it is a map of objects that know their keys).
I think the way forward is to "modernize" the API of Dictionary<K,V> if it would help performance (eg maybe https://github.com/dotnet/corefx/issues/31598) and then try to evolve to something as fast as possible. Thoughts from others?

@danmosemsft

I assume you mean on the comparers, because you made FindEntry and RemoveEntry virtual. For the comparer, you are calling through an interface.

I was not intending for FindEntry and RemoveEntry to be virtual; is there something that implies virtual that I am not aware of?
The comparer interface is applied as a generic type parameter parameter, so that a struct comparer implementation can be used to get the compiler to specialize, de-virtualize, and likely inline as well without too much API compromise.

Any particular reason? Some collision resolution strategies require a primal size, right?

Modulo prime is a pretty good strategy for handling hash keys with poor entropy in the low bits (memory addresses of objects, for example). The problem is it it requires a DIV instruction with a best case latency of several dozen cycles, which is an unwelcome price to pay when you've already got good low bit entropy. A less resilient but faster entropy mixing function like multiplying by<PHONE_NUMBER> (2^32/phi) can be used, or left to the consumer.

Do you mean a bidirectional map? That seems something to worry about later. (If by PropertyDictionary you mean this, then it is not a bidirectional map, it is a map of objects that know their keys).

I mean a map of objects that know their keys (which at least in my use cases means a set of values/objects with an "index" of some trait for fast lookup).
I would like a bidirectional map too, but I think it would have to be a different structure.

@safern


NSM forwarders can't be partially instantiated

We should solve this when solving #32534.

Chatted with @sjindel-google, this is related to a larger issue with instantiation of local and top level method (https://github.com/dart-lang/sdk/issues/31665). This issue does not have a milestone associated with it. We should probably set it to 'dart2 beta 4' or 'Dart2 Stable' and adjust the related issues to the same milestone. We have moved this to 'Dart2 Stable' for now.

This is solved by https://dart-review.googlesource.com/c/sdk/+/53801, which moves handling of partial instantiation into the shared prologue builder.

This is actually still a problem:
$ echo >nsm_partial.dart <<EOF
class C {
  T foo<T>(T x);

  dynamic noSuchMethod(Invocation invoke) {
    print(invoke.typeArguments);
  }
}

void main() {
  var c = new C();
  int Function(int) k = c.foo;
  k(1);
}
EOF
$ pkg/vm/tool/dart2 nsm_partial.dart
[]


Is "karpenter.sh/do-not-evict" documented correctly?

Is an existing page relevant?
https://karpenter.sh/v0.22.1/concepts/deprovisioning/#consolidation
What karpenter features are relevant?
Pod Eviction.
How should the docs be improved?
I see this text under "Pod Eviction".
spec:
  template:
    metadata:
      labels:
        karpenter.sh/do-not-evict: ""

The go code, scheduling.go, has this:
func HasDoNotEvict(pod *v1.Pod) bool {
	if pod.Annotations == nil {
		return false
	}
	return pod.Annotations[v1alpha5.DoNotEvictPodAnnotationKey] == "true"
}

Should the documented annotation be
spec:
  template:
    metadata:
      labels:
        karpenter.sh/do-not-evict: true

?
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Seems like true is the expected value.

Fixed in https://github.com/aws/karpenter/pull/3168


For some reason the builds hang

Needs to be debugged

https://travis-ci.org/link0/profiler/jobs/53700902

Fixed in https://github.com/link0/profiler/commit/956f482687a687c5e9d8f7fe30fa8f391a677692


Website does not have the correct trademark disclaimer

As part of our ongoing effort to https://github.com/cncf/techdocs/issues/198, we noticed that the website does not pass the trademark criteria on CLOMonitor.
To fix this:
Head to the source code of the website. In the  section, add a disclaimer or link to the Linux foundation trademark disclaimer page:
Disclaimer

The Linux Foundation has registered trademarks and uses trademarks. For a list of trademarks of The Linux Foundation,
         please see our Trademark Usage page.

Link

Trademarks

It looks like there is something wrong with the detection script. The disclaimer is there on the website:

The issue here is that there's a forward slash appended to the url:
https://www.linuxfoundation.org/trademark-usage/.
The correct url should be:
https://www.linuxfoundation.org/trademark-usage

Sorry this one fell through. I'd prefer to keep the url pointing to the latest release rather than the in-dev version if possible. Is it possible to override the site CLOMonitor checks? If not, how about simply adding the disclaimer as well to the redirect page?


Refactor Environment

Our current environment system is pretty sub-par or just convoluted. We should really rewrite it using a "dotenv" approach -- which it currently does but not efficiently enough. A good reference: https://github.com/vlucas/phpdotenv
Tasks

[x] Remove Host and fallback stuff
[x] Remove Bootstrappers
[x] Remove hostname matching
[x] Require no nginx/apache env vars
[x] Add a .env finder and parser
[ ] Must be immutable
[x] Should also use putenv()

Possible

[ ] Rename package to Env?
[x] Remove events?

Thoughts @exonintrendo ?

If the environment class handles environment/system specific variables (such as hostname, credentials, etc.), would we have it solely read from $_ENV and $_SERVER? Or should we allow for the presence of some file/config override to set desired values? Just considering multiple applications running on one machine, they may reference different database credentials, hostnames (virtual host), etc.

Here's the current branch. https://github.com/titon/framework/tree/env-redo/src/Titon/Environment
Right now, it would find and load an .env file that contains key-values (in an INI compatible format).
$env = new Titon\Environment\Detector('/path/to/configs');
$env->initialize();

APP_ENV = local
DB_NAME = titon
DB_USER = titon
DB_PASS = "hsadYGBQ@93(by9s9h7jk"

And since $_ENV and $_SERVER are immutable, they would have to use getenv() or the Titon specific env() function.
env('APP_ENV'); // local
getenv('DB_USER'); // titon

Ah, gotcha. Dunno why I didn't see the branch. Sounds good to me!

Once they are loaded, they will by immutable, so yes. I just haven't finished that yet.


Deploying the app using PM2 in EC2

Hi,
I am following this tutorial https://medium.com/@Keithweaver_/setting-up-mern-stack-on-aws-ec2-6dc599be4737 and trying to deploy this app.
However, I am stuck in this step pm2 start server.js -- --production . It is throwing error but there is no description of error / logs.
Have anyone deployed this app to EC2 with PM2 ? Can you please help in deploying this app ?
Or is there anything wrong with this command pm2 start server.js -- --production
Thanks,

I just deployed on DigitalOcean droplet and this worked.
I reckon it should work for EC2 as well.
start:prod script under packages.json gave me some clues
"start:prod"  :  "cross-env NODE_ENV=production node index.js"
According to this we need to set environment as production and then run index.js
pm2 config and setup
Step 1:
cd to project folder
Step 2: Create ecosystem.config.js file for pm2
with the following config
module.exports = {
  apps : [{
    name      : 'MERN',
    script    : 'index.js',
    env: {
      NODE_ENV: 'development'
    },
    env_production : {
      NODE_ENV: 'production'
    }
  }],
};

Replace 'MERN' with your app's name so it will show in pm2 list.
Step 3: Start app's process using pm2
For production -
pm2 start --env production
For development -
pm2 start --env development
You should see following output
 App name │ id │ mode │ pid   │ status │ restart │ uptime │ cpu  │ mem       │ user │ watching │
├──────────┼────┼──────┼───────┼────────┼─────────┼────────┼──────┼───────────┼──────┼──────────┤
│ MERN     │ 0  │ fork │ 29305 │ online │ 0       │ 3m     │ 0.1% │ 93.4 MB   │ root │ disabled │

Thats it!  👍  Your app is  now up and running.
Hope this helps!
Please let me know how it goes.
Some basic pm2 operations / [ replace MERN with your app name]
Stop App
pm2 stop MERN
Start App
pm2 start MERN
Monitor App
pm2 monit MERN
Delete App
pm2 delete MERN
Show list of running pm2 processes
pm2 list

Hi,
Thanks for the response. Let me spin up an instance with all these changes and I will update if it worked. Once again, thanks.

Will close for the time being, feel free to reopen if there is still an issue related to the repo itself.


sync local paths of attached files and automatically upload attached files

Hi,
Thank you for your work. I have used zotero for a long time. Usually, I add papers with pdf files to zotero using its Chrome extension. When I come to notion, I find notion can not link a local file. That causes a lot of trouble. Could you add local paths of attached files in sync list? If it can automatically upload attached files, that will also be great.
Advice:
Since people may have different needs on sync information, I advise to set a sync list so that users can choose the items they want to sync from zotero.
Thank you！

Hi @hengfei-wang, thanks for the feedback!
I've also found that Notion does not have a way to link to local files. It also can't link to local Zotero items using its zotero protocol (e.g. zotero://select/library/items/XN4SL8R5) which I was originally hoping to use to link directly from Notion to the Zotero desktop app. Notion can only successfully link to the https URLs pointing to the Zotero web app (e.g. https://www.zotero.org/users/8888888/items/XN4SL8R5).
The Notion API does not currently support uploading files, so I think the best we could do for now is have a text property in Notion with the path to the local file. You'd then have to manually copy that path and open it on your system in some manner (perhaps pasting into your browser). Would that be beneficial at all?

Thank you for your reply.
Yes, it is totally fine if you could add a text property including local path. I can set a shortcut in alfred to open a file from selected text.

@hengfei-wang file path syncing has just been released in version 0.2.1! You can get the file paths into Notion by adding a new Text property named File Path to your database and then syncing your items again.
Let me know how this works for you!


Parameter crop = false, still crop image

When passed false to parameter crop (getPictures(false) ), this not showing resize area, but still crop image.
I understand that if false not showing resize area and no auto crop.

Thank you so much @nilotpalkapri, your PR resolved this issue 😀


[Suggestion] Broadcast the exact coords of an envoy.

Hi, I have the suggestion to add an option to broadcast the exact coords of an envoy on chat, this is useful for random envoys in a big world.

Added to our tracking board.


No transaction history

From Copay:
Updating Transaction History
Syncing TXs. Got:0 Skip:5 EndingTxid: null Continue: undefined
Finished Sync: New / soft confirmed Txs: 0
Tx History synced. Total Txs: 0
Tx History saved.

This 2-of-4 watch-only wallet has plenty of transactions. There are no errors in the log.
What causes this?

Are they dust?

No, there are many and quite large transactions


fix train epoch metrics

What does this PR do?

fix #3228
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @beyondyoni! Thanks for updating this PR.

In the file pytorch_lightning/trainer/connectors/logger_connector.py:

Line 164:44: E231 missing whitespace after ','

@williamFalcon Thanks for assisting!
as you see, I only made 2 very small changes. For some reason I'm trying to understand, they cause the CI tests to crash

yes lol... this is what i'm saying... it's not as simple as just 2 changes.
This is a much deeper thing that touches a lot. I'm afraid this has to wait until after 1.0.
We can't add the instability to the package.


auto increment version in bower.json

There has been no activity on this issue for several months and that's why are closing it (automatically). If you think this still needs to be addressed please reopen it. The good news is that we are now working on the next version of CARTO.js, stay tuned.


script tag needs type="module" for Sveltia CMS to appear in Astro project, but README doesn't say so

I have just tried to migrate my Astro project from DecapCMS to Sveltia CMS, following the instructions. However, the viewport remained empty when navigating to the admin route.
Through the zankhq/astro-starter template I found out that the script tag needs type="module" for Sveltia CMS to work under these circumstances. This is not mentioned in the README.
I don't know when type="module" is needed and when it isn't, but I wanted to report back this papercut in case the maintainer(s) want to improve the instructions.

sveltia-cms.js was shipped as an ES module in earlier versions so type="module" was needed. v0.21.0 changed the bundle to an IIFE so you no longer need it.


Switch to Whitelist, Instead of Blacklist, for camera2

It seems like the camera2 implementations are about the same great big steaming pile of unimpressive code that the original Camera implementations are.
With that in mind, revisit forceClassic() and the current camera2 blacklist and switch to more of a whitelist model. Devices that I can vet as supporting camera2 use it; otherwise, go with Camera.

Documentation for this needs to cover the deprecated/unused force-classic API, the new force-engine API, and how to add to the whitelist from the app via DeviceMatcher.

Addressed in v0.6.0.


Add contact me section

@Janhvi52  can you please assign this issue to me I want to work on it under kwoc2022

@Janhvi52  can you please assign this issue to me I want to work on it under kwoc2022

Sure go ahead @Riya14-dangra  but make sure you have on right side of page the name,email and message in the form and on left side email-id and phone no and after writing the message it should get updated in an excel/spreadseet the details of the person and finally shows message sent successfully!
You got it right?

@Janhvi52  can you please assign this issue to me I want to work on it under kwoc2022

Sure go ahead @Riya14-dangra but make sure you have on right side of page the name,email and message in the form and on left side email-id and phone no and after writing the message it should get updated in an excel/spreadseet the details of the person and finally shows message sent successfully!
You got it right?

@Janhvi52 please have a look Hope you are expecting in these same way

Peace! just look forward to ,"message sent successfully" upon pressing the submit button and if possible maintain an excel sheet/spreadsheet which keeps track of the details of whoever filled out the form
thanks :)

@Janhvi52  can you please assign this issue to me I want to work on it under kwoc2022

Sure go ahead @Riya14-dangra but make sure you have on right side of page the name,email and message in the form and on left side email-id and phone no and after writing the message it should get updated in an excel/spreadseet the details of the person and finally shows message sent successfully!
You got it right?

@Janhvi52 please have a look Hope you are expecting in these same way

Peace! just look forward to ,"message sent successfully" upon pressing the submit button and if possible maintain an excel sheet/spreadsheet which keeps track of the details of whoever filled out the form thanks :)

Okay I will Try for it
Should I move forward to contribute @Janhvi52

@Janhvi52  can you please assign this issue to me I want to work on it under kwoc2022

Sure go ahead @Riya14-dangra but make sure you have on right side of page the name,email and message in the form and on left side email-id and phone no and after writing the message it should get updated in an excel/spreadseet the details of the person and finally shows message sent successfully!
You got it right?

@Janhvi52 please have a look Hope you are expecting in these same way

Peace! just look forward to ,"message sent successfully" upon pressing the submit button and if possible maintain an excel sheet/spreadsheet which keeps track of the details of whoever filled out the form thanks :)

Okay I will Try for it Should I move forward to contribute @Janhvi52

sure waiting for the outcome!


Create Folder and File

ISSUE: #5
Created a file and folder by terminal

Hey @dhruvgoyal-098
Thanks for opening this PR 🚀. Mentor will review your pull request soon and till then, keep contributing and stay calm.
Thanks for contributing in OpenCode'24 ✨✨!


Install-WindowsUpdates.ps1 is failing to install April 2024 updates

Description
Windows April 2024 security update (KB5036909) fails to install.

Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] Windows Server 2019
[X] Windows Server 2022

Image version and build link
latest version
Is it regression?
Worked before patch April 9, 2024
Expected behavior
Updates should install successfully or at least image generation should complete
Actual behavior
Updates do not install, image generation fails
Repro steps
Trigger a build and wait

Hello @frede192!
As you can see, Windows22 test build on the main branch was successful yesterday. Please make sure you are using the main branch and correct configuration.

Hi @MaksimZhukov,
I just checked. We are using the main branch, and have followed these docs as part of setup:
https://github.com/actions/runner-images/blob/main/docs/create-image-and-azure-resources.md#automated-image-generation
Our packer build command which results in a failed windows update looks as follows:
packer build -var "client_id=$ClientId"
-var "client_secret=$ClientSecret"                 -var "install_password=$InstallPassword"
-var "location=$Location"                 -var "subscription_id=$SubscriptionId"
-var "temp_resource_group_name=$TempResourceGroupName"                 -var "tenant_id=$TenantId"
-var "virtual_network_name=$VirtualNetworkName"                 -var "virtual_network_resource_group_name=$VirtualNetworkRG"
-var "virtual_network_subnet_name=$VirtualNetworkSubnet"                 -var "managed_image_name=$ImageType-$ResourcesNamePrefix"
-var 'azure_tags={ "ExcludeMdeAutoProvisioning": "True" }'                 -var "managed_image_resource_group_name=$ResourceGroup"
-color=false                 $TemplatePath

Hi again @MaksimZhukov,
I tried to re-run the pipeline (without doing any changes) and today it works.
So I do not know why 'Install-WindowsUpdates.ps1' failed yesterday when trying to install security update (KB5036909).
I will close this issue for now. If run into this problem again in the future, then we can revisit it then.
Thank you


uucore: turn backup_control into a feature

This PR turns backup_control into a feature. It is related to https://github.com/uutils/coreutils/issues/5202

Super cool that you're picking this all up! Thanks!


Create binder styles tests for pulse

What is the expected enhancement?
As part of #6467, some old pulse image comparison tests that were being skipped were removed. These were in the file test_pulse_visualization_output.txt. The tests in this file may be useful when a #4544 binder style set of image comparison tests is created for pulse.

Dear @enavarro51 ,
Dear @taalexander ,
I'd have a look at this, if possible.

hey @MartinBeseda sorry for the slow response, are you still interested in working on this? If yes lmk and I'll assign to you 😄

Hey, @javabster can I work on this?

Hi @ozamanan as @MartinBeseda is unresponsive I will assign to you, let us know if you have any questions 😄

Any news @ozamanan ?

Hi @anonymousr007
The assigned contributor is unresponsive are you interested in taking over?

Thanks @AbdullahKazi500 ! assigning.
Small headsup. The binder mechanism is growing intro proper CI testing in https://github.com/Qiskit/qiskit-terra/pull/9961 . There are two options at this point:

You can write pulse tests top of AngeloDanducci:ad9892. This creates a dependency and a potential blocker, but it does not need follow up.
You can do it in the notebook way and then we moved them to the CI way. This allows you to work more independently, but it might requiere an update if 9961 lands before yours.

Up to you. Whatever you feel more confortable with.

Hi @1ucian0! It seems  #9961 is merged, binder tests are integrated in the workflow.  Is this issue still open ?

Hi @Raghav-Bell Yeah I am working on this for now

Hi @AbdullahKazi500 How is that going?


Deploy releases to maven repo (https://mvnrepository.com/)

I know of https://maven.quiltmc.org/repository/release/org/quiltmc/quiltflower/ but the number one source for deployed dependencies is Maven Central ~the Maven Repository https://mvnrepository.com/~.
Please deploy releases to this repository.

mvnrepository.com is not a repository, it's an indexer. I believe the team has already begun plans to publish releases to maven central via OSSRH though, it's just a matter of ensuring the appropriate keys are configured in the appropriate places and it's set up in a way that's sustainable for the organization.

You are right. I meant Maven Central.

Hi, thank you for opening an issue, and sorry for the delay! The latest release version (1.9.0) is now on Maven Central for public use. Thanks!


Avoid lbryinc calls before auth_token (stub)

@seanyesmunt and @jessopb understand

waiting for sean's changes


feat: Generate ESLint globals file for auto-imports

This closes #448. A new file, .wxt/eslintrc-auto-import.json is generated when the .wxt/ directory is generated (prepare, dev, build, zip, etc).
If you have ESLint installed already, the JSON file will be generated automatically. You just need to extend it in your ESLint config:
// .eslintrc.js
module.exports = {
  extends: [
    './.wxt/eslintrc-auto-import.json',
  ],
}

If the file isn't being generated (eslint isn't a direct dependency, or you're in a monorepo and it's not a dependency of the package at all), you can enable this file manually:
// wxt.config.ts
export default defineConfig({
  imports: {
    eslintrc: {
      enabled: true
    },
  },
})

Released in v0.16.7


Pancreas ontology expansion for colouring of anatomogram shapes/structures

The ontology expansion and so the colouring of parent structures for cell types across images in the pancreas anatomogram stack is a bit patchy:
In none of the experiments below 'pancreatic acinus' structure/shape is active (grey colour) although the data include acinar cells, which is a child term for pancreatic acinus: https://www.ebi.ac.uk/ols/ontologies/efo/terms?iri=http%3A%2F%2Fpurl.obolibrary.org%2Fobo%2FCL_0000622
screenshot here:
screenshot.pptx
https://wwwdev.ebi.ac.uk/gxa/sc/experiments/E-GEOD-81547/results/anatomogram
https://wwwdev.ebi.ac.uk/gxa/sc/experiments/E-MTAB-5061/results/anatomogram
https://wwwdev.ebi.ac.uk/gxa/sc/experiments/E-ENAD-27/results/anatomogram
This one doesn't have either the acinus or the islet of Langerhans in grey, which is especially weird as the expansion from islet cell types to islet (structure) and the colouring of these across the two images works in the other pancreas experiments
https://wwwdev.ebi.ac.uk/gxa/sc/experiments/E-MTAB-5061/results/anatomogram

Thanks @sfexova, I suspect that this might be fixed when we move to the newer version of solr cloud and to the new version of the ontology (both things should happen at the same time). This was delayed because the loading of data with this was failing. We keep this open though.

Thanks @sfexova, I suspect that this might be fixed when we move to the newer version of solr cloud and to the new version of the ontology (both things should happen at the same time). This was delayed because the loading of data with this was failing. We keep this open though.

Yes, but I don't think that can explain all of it, really. These are connection present in the 'old' ontology (EFO) already and Alfonso said we're still using that one for this release. Furthermore, the same connection appears to be 'seen' by some experiments but not others:
all of the experiments above share most of the islet of Langerhans cell types (pancreatic A cell, pancreatic D cell, type B pancreatic cell) and yet in here: https://wwwdev.ebi.ac.uk/gxa/sc/experiments/E-MTAB-5061/results/anatomogram
not even the islet of Langerhans (the roundish blob next to the branchy structure) is highlighted in the pacreas main picture. And that makes me also think - why the anatomograms can pick up this one connection from this ontology and not the other from the same one (e.g. acinar cell >> pancreatic acinus) - but yes, I gues that could be Solr??

Yes, but I don't think that can explain all of it, really. These are connection present in the 'old' ontology (EFO) already and Alfonso said we're still using that one for this release. Furthermore, the same connection appears to be 'seen' by some experiments but not others:
all of the experiments above share most of the islet of Langerhans cell types (pancreatic A cell, pancreatic D cell, type B pancreatic cell) and yet in here: https://wwwdev.ebi.ac.uk/gxa/sc/experiments/E-MTAB-5061/results/anatomogram
not even the islet of Langerhans (the roundish blob next to the branchy structure) is highlighted in the pacreas main picture. And that makes me also think - why the anatomograms can pick up this one connection from this ontology and not the other from the same one (e.g. acinar cell >> pancreatic acinus) - but yes, I gues that could be Solr??

Silvie’s right, this is a problem in the code. We’re not collecting parents and other relationship accessions (part_of, located_in) to colour the anatomogram, only the field ontology_annotation. I’ve created a ticket associated this issue.

Great that you found the source of the error @alfonsomunozpomer.

@sfexova can you check whether this is still an issue against the current wwwdev with the new v5 collections? Thanks

yes - the islet of Langerhans is now highlighted in the main picture as it should be (but I that was just some weird glitch in one of the experiments as it was OK in the others even before but the main issue) I was drawing attention to, with the acini (that were never working as expected in any of the pancreas experiments we have), is still there - see attached pic for details/explanations, example taken from https://wwwdev.ebi.ac.uk/gxa/sc/experiments/E-MTAB-5061/results/anatomogram but the same issue appears in the other pancreas experiments too
pancreas_ontology_info_propagation_issues.pdf


Wellness Encounters [Record_Height [Observation]]

Looks to me like these ought to be VitalSign states and not Observations. According to the Wiki, only vital signs have the vital_sign field.

Also some in metabolic_syndrome_disease and metabolic_syndrome_care

I changed these Observations to VitalSigns, but many have no exact or range, so I assume these are not required by Synthea. I'm guessing these vitals are managed by the Ruby code and having no exact or range means to use the managed values instead?
"The VitalSign state type indicates a point in the module where a patient's vital sign is set. Vital Signs represent the physical state of the patient, in contrast to Observations which are the recording of that physical state."

Observations are allowed to have the vital_sign field - that indicates that instead of directly specifying a value or range of possible values, the value of the corresponding VitalSign is recorded in the Observation. You're right that the vital signs in question are managed by the ruby code (specifically  https://github.com/synthetichealth/synthea/blob/master/lib/modules/lifecycle.rb#L165 ).
Ideally I'd like to have as many vital signs as possible represented in the GMF instead of in the ruby code, but many vital signs have complex formulas, which we haven't found a good way to represent in the GMF yet.

Ok, I see that now. Was it there before? I can't imagine that I missed it.

I've implemented vital_sign in Observation but now I'm finding situations where no vital sign was recorded before the observation requested it (e.g. Height). What's the recommended implementation?

Some of the vital signs are assigned at birth so that doesn't happen in our system.

Thanks, I do that too but I was just wanting a clarification. For signs like height and weight, I see you have assigned trajectories for them. Height levels off around 21 and weight increase slows down after then (depending upon diet and exercise risks)?

Yes, that is right. Weight also stabilizes at age 49 then starts to decrease at 60. Those are configurable, and there are citations in the config/synthea.yml file.

Closing this issue as it has been resolved


ai.djl.engine.EngineException: Failed to load PyTorch native library

Exception in thread "AWT-EventQueue-0" ai.djl.engine.EngineException: Failed to load PyTorch native library
	at ai.djl.pytorch.engine.PtEngine.newInstance(PtEngine.java:85)
	at ai.djl.pytorch.engine.PtEngineProvider.getEngine(PtEngineProvider.java:40)
	at ai.djl.engine.Engine.getEngine(Engine.java:186)
	at ai.djl.Model.newInstance(Model.java:99)
	at ai.djl.repository.zoo.BaseModelLoader.createModel(BaseModelLoader.java:189)
	at ai.djl.repository.zoo.BaseModelLoader.loadModel(BaseModelLoader.java:152)
	at ai.djl.repository.zoo.Criteria.loadModel(Criteria.java:168)
	at ai.djl.repository.zoo.ModelZoo.loadModel(ModelZoo.java:141)
	at Yolo.yolo(Yolo.kt:53)

Caused by: java.lang.UnsatisfiedLinkError: /Users/Smile/.djl.ai/pytorch/1.13.0-cpu-osx-x86_64/0.20.0-libdjl_torch.dylib: dlopen(/Users/Smile/.djl.ai/pytorch/1.13.0-cpu-osx-x86_64/0.20.0-libdjl_torch.dylib, 1): Symbol not found: __ZNSt3__113basic_filebufIcNS_11char_traitsIcEEE4openEPKcj
  Referenced from: /Users/Smile/.djl.ai/pytorch/1.13.0-cpu-osx-x86_64/0.20.0-libdjl_torch.dylib (which was built for Mac OS X 12.0)
  Expected in: /usr/lib/libc++.1.dylib

System: MacOS 10.15.7 (19H2026)
It looks like my libc++.1.dylib is missing some symbols? I'm curious, why doesn't your libs come with libc++ or use a static dependency on libc++? How should I proceed now, is there any remedy, I can't upgrade my computer system version and I can't modify the system libc++. please feel free to let me know if you need to provide any information.

@o2e
PyTorch 1.13.0 was built on mac 12, mac 10 is not supported any more. You can either upgrade your mac or use lower version of PyTorch.


use Vector API to read/write primitive arrays

Summary
Use Vector API to read/write primitive arrays.
Checklist
Delete items not relevant to your PR:

[x] Unit and integration tests covering the common scenarios were added
[x] A human-readable description of the changes was provided to include in CHANGELOG
[ ] For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

Tried VarHandle, Foreign Memory & Vector APIs, but the performance improvement was less than 10% when using JDK 21-ea.26. Will revisit this after JDK 21 is officially released.


Version bump and details for 3.3.0

Ping @ryanpetrello, @john-westcott-iv

Coverage remained the same at 64.947% when pulling 1dbf24dfe8f8c22d8c37f682d30e4ea7a81f54e3 on AlanCoding:release33 into cec84135376624e8d3fdf1f83eac01620f7b7b25 on ansible:master.


Migrating from 0.9.6? API uses completely different syntax

Hey all - I've been using this API (and loving it!). Discord was rejecting my bot's connection, starting last night, citing "code 4012: Requested gateway version is no longer supported or invalid". I updated the API to it's current version 1.0.1 and my entire bot is throwing errors left and right. It appears the entire syntax/how the API library works has been completely... changed. The bot is sort of essential to our work and I'm immensely confused by all the changes to the API. I located the "migrating from 0.9x guide", but this new syntax is waaaaay over my head.
Is there anywhere in the documentation where I can see examples of sending messages to a channel via ID (located in the same server the bot is in), and editing messages? I tried all the different types of messages but none of them seem to support editing, aside from one which requires the string to be a task instead of a string?
I'm really lost here, and a few code samples would go a long way. Thanks.
Alternatively, is there any way I can use the 0.9.6 API and still access discord like before?

no discord has stopped APIV5 the one 0.9 used your only option is to start over with 1.0.2 (1.0.1 is broken)
feel free to join https://discord.gg/discord-api and look for #dotnet_discord-net

That's unfortunate, but I understand. I'll go ahead and remake it, better and faster, stronger, etc. Thanks for the help, closing this issue!

If it helps there's a lot of public bots using the new Syntax, and most of the time you can find what you're looking for by searching the channel's messages


sql blog

#514

I'm fine pushing this live, but will put in another issue. This gets me intrigued, but not productive. We need an example that walks through the states, token management, and other goodies. The one change that I think is necessary is to change now is the canonical URL.

we can create a separate issue for creating a sql agent example to show you how to use it, but this ticket is scoped to be an intriguing blog article.
I will rename this page to agentic_sql. let me know if you think another top level page would work better.

Thanks Luke. Agentic_sql is a great page name. I think we had different
interpretations of the word docs. We know for next time. I hope you're
enjoying your travels!
On Tue, Oct 1, 2024, 5:42 AM Luke Lalor @.***> wrote:

I'm fine pushing this live, but will put in another issue. This gets me
intrigued, but not productive. We need an example that walks through the
states, token management, and other goodies. The one change that I think is
necessary is to change now is the canonical URL.
we can create a separate issue for creating a sql agent example to show
you how to use it, but this ticket is scoped to be an intriguing blog
article.
I will rename this page to agentic_sql. let me know if you think another
top level page would work better.
—
Reply to this email directly, view it on GitHub
https://github.com/eidolon-ai/eidolon/pull/819#issuecomment-2385309412,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACS2HZ4C5BUSJAASGR5QFITZZJVAZAVCNFSM6AAAAABPDCQJMOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBVGMYDSNBRGI
.
You are receiving this because your review was requested.Message ID:
@.***>

Thanks Luke. Agentic_sql is a great page name. I think we had different interpretations of the word docs. We know for next time. I hope you're enjoying your travels!
…
On Tue, Oct 1, 2024, 5:42 AM Luke Lalor @.> wrote: I'm fine pushing this live, but will put in another issue. This gets me intrigued, but not productive. We need an example that walks through the states, token management, and other goodies. The one change that I think is necessary is to change now is the canonical URL. we can create a separate issue for creating a sql agent example to show you how to use it, but this ticket is scoped to be an intriguing blog article. I will rename this page to agentic_sql. let me know if you think another top level page would work better. — Reply to this email directly, view it on GitHub <#819 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACS2HZ4C5BUSJAASGR5QFITZZJVAZAVCNFSM6AAAAABPDCQJMOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBVGMYDSNBRGI . You are receiving this because your review was requested.Message ID: @.>

@flynntsang sounds good. Docs in the original ticket referred to making sure that it is documented on our website. IE the automated documentation. That is enough to get somebody who is willing to truck through some pain started. An example/walkthrough then makes that process as easy as possible.


Replaced deprecated function from sfa/base.py

The pandas.Series.nonzero function is now deprecated (see https://pandas-docs.github.io/pandas-docs-travis/reference/api/pandas.Series.nonzero.html). It is replaced by numpy.nonzero (https://numpy.org/devdocs/reference/generated/numpy.nonzero.html) and the recommended course of action is to convert the Series into a numpy object.

Thanks for your contribution :)


An issue when using Link and ScrollOverPack

Hi there,
As I saw in your example, you fix the height of each section and that works great.
However, in my case, I cannot fix the height because my content will increase the height of the section when open in mobile device.
If I don't use a ScrollOverPack, then it works fine, but I want to use it too.
I realize that the TweenOne elements will be not shown until I scroll to that section. That makes Link scroll to the wrong position.
Do you have any idea?
Best,

https://github.com/react-component/scroll-anim/pull/12

0.4.0

0.4.1
offsetTop={58} not is negative.


FooterBrands component

Epic https://github.com/x-team/auto/issues/573

Create new component.

Dependencies

Graphic component https://github.com/x-team/auto-ui/issues/13

Props
brands: [
  {
    title: string,
    image: string,
    url: string,
  }
]

Behaviour
None specific behaviour.

@bernardodiasc do you think we need handleClick for each brand? Or would url suffice?

Yeah, this also could work using url and rendering with <a>. :+1:

Something I've seen in other sites with list of brands - have the logo "darken" on hover, possibly by applying a css transition on the opacity.


Add link to Scylla University from the standard theme

@tzach In the footer, next to "Docs"?

Yes, we can try that.
Second alternative is on the top menu.
@annastuchlik what do you think?

I opt for the footer, as the top menu is product documentation-oriented.


Thoughts on LoadInitState

The goal of LoadInitState is to allow the following script to repeatedly run without modification to continue training:
train_end_cp = TrainEndCheckpoint(dirname="exp1")
load_init = LoadInitState(train_end_cp)
net = NeuralNet(..., callbacks=[train_end_cp, load_init])
net.fit(X, y)

I have been wondering if adding raise_if_error to load_params would be better:
train_end_cp = TrainEndCheckpoint(dirname="exp1")
net = NeuralNet(..., callbacks=[train_end_cp])

net.initialize()
net.load_params(checkpoint=train_end_cp, raise_if_error=False)
net.partial_fit(X, y)

The goal of LoadInitState is to allow the following script to repeatedly run without modification to continue training:

Could you explain what needs to change for this? I made a quick check and this worked for me as expected (i.e. training continued from where it left off), I didn't see any error.

The first snippet already works, no changes needed.
I want to see what you guys think of the second approach for doing the same thing. Most likely there is nothing wrong with the first snippet and this is a non-issue.

I think I get what you mean. If there is no checkpoint yet, load_params would fail. Adding a flag to ignore the error would allow to use less code for both situations.
I dislike the proposal since it doesn't convey the meaning of the code as the LoadInitState variant does. The latter makes it clear what is happening and why. The former adds a dubious flag for ignoring errors where the intention is not clear at all.
Before we search for alternatives to the LoadInitState callback I'd rather gather reasons for its existence and identify possible shortcomings. One thing that justifies the existence of LoadInitState is, for example, that it is quite easy to accidentally re-initialize the network upon setting parameters. If there was a way to 'safely' re-initialize parameters, loading an initial state upon every fit would not be necessary. Any other reasons?

@ottonemo You make good points. After some more thought, I do not think my proposal is better than the current approach. I was trying to search for alternatives to see if there was an approach with a better API design.
Given everyones feedback, the current design with LoadInitState looks pretty good.


Identity Provider behind a Proxy

I have a project where FE is hosted publicly but all BE services including Identity Server are living behind a publicly available reverse proxy. The reverse proxy URL is something like https://proxy.com.
When FE is calling Identity Server API through the Proxy via https://proxy.com/identity-server-public it's possible to get the JWT token. Even more the issuer is automatically set to https://proxy.com/identity-server-public by Identity Server.
However, when JWT token is received by FE and then sent to BE services through the reverse proxy, it fails authorization since internal services are calling Identity Server via the internal address - http://identity-server-private.com (it's configured as issuer) so the token's issuer from JWT token is not matching.
We can not use http://identity-server-private.com from FE since it's not reachable from the outside, and we can not use https://proxy.com/identity-server-public from internal services since it's not reachable from the inside (all services are living in private network).
Is there a way to configure angular-oauth2-oidc library to send requests to https://proxy.com/identity-server-public but use http://identity-server-private.com as a valid issuer? Or maybe you can suggest anything else that could help?

@zapling Actually I found a solution that works for us. When configure OAuth configs you can set:
strictDiscoveryDocumentValidation: false (https://manfredsteyer.github.io/angular-oauth2-oidc/docs/additional-documentation/using-an-id-provider-that-fails-discovery-document-validation.html)
What will allow you to set issuer in OAuth configs to one domain but send auth http requests to another domain 🙂

@rustam-ashurov-mcx Thanks for the quick response :) The following fields in the AuthConfig seems to have solved the validation issues for me
	skipIssuerCheck: true,
	strictDiscoveryDocumentValidation: false,


Support windows-amd64

Hi! great tool, I tried cargo build and it seems to build in windows. Would you like to provide a windows binary too?

Hi, xplr isn't supported on Windows. This extension is only a wrapper that calls xplr, so, even it compiles, unfortunately it still won't work.
You will need WSL to run both.


Not possible to toggle options in useComments

Describe the bug
It's currently not possible to toggle options like commentsOfOrdering and commentsRankingFilter  in useComments. can this be supported?
Edit: in fact, these attributes don't seem to be passed to the actual graphql query at all.
Expected behaviuor
i expect those options to be toggled so the user can toggle the order of comments in the ui.
If using the React bindings, what is the essence of your React app?
react-web 1.2.2 and 1.3.0-next.4

They can easily be added.

@moesalih this is now available in 1.3-next.8 can you please update and confirm works as intended so we can close this issue?


[v0.3 DeNas] fix DeNas README and add PyTorch1.12 Dockerfile

provide PyTorch1.12 for DeNas
fix other docker related issues and README
fix DeNas configuration files

@zigzagcai , I saw you changed Dockerfile name, please also update https://github.com/intel/e2eAIOK/blob/main/scripts/start_e2eaiok_docker.py#L291-L317

@zigzagcai , I saw you changed Dockerfile name, please also update https://github.com/intel/e2eAIOK/blob/main/scripts/start_e2eaiok_docker.py#L291-L317

Got it. I will update it accordingly.

@zigzagcai , it would be better to put utils in common lib since the function can be widely used.

Got it. I will fix it.

@zigzagcai , it would be better to put utils in common lib since the function can be widely used.

It has been fixed.

@zigzagcai , I saw you changed Dockerfile name, please also update https://github.com/intel/e2eAIOK/blob/main/scripts/start_e2eaiok_docker.py#L291-L317

I have updated the corresponding start_e2eaiok_docker script.

We provide latest PyTorch1.12 for DeNas, and it works well with DeNas code.
Development completed.

@xuechendi Could you please review and merge this PR. All comments has been fixed.
DeNas related DockerfilePytorch120 works well.

@xuechendi Could you please review and merge this PR. All comments has been fixed. DeNas related DockerfilePytorch120 works well.

@zigzagcai , hi, zheng, I found you changed backend name in start_e2eaiok_docker,py, which makes it incompatible with input argument(https://github.com/intel/e2eAIOK/blob/fa0911113af01785c485bd3c0d48b7c5dbec78cb/scripts/start_e2eaiok_docker.py#L15) and readme for dlrm
Can you keep backend name stays the same?

@zigzagcai , I added some comment to your latest fix, If pytorch120 is working for all pytorch model except dlrm, you can remove pytorch110, but fix RNNT readme accordingly.

@zigzagcai , I would like you to own start_e2eaiok_docker.py, this is our click-to-run install/deploy entry for all models. So make sure start_e2eaiok_docker.py can always do one-click-run to deploy docker env for all models.(include v0.1, v0.2 and denas)
We may need to do manual check for now, but it would be great if we can add it to CI/CD later. Thanks

PyTorch1.12 base image has bug with torchccl

@zigzagcai , I added some comment to your latest fix, If pytorch120 is working for all pytorch model except dlrm, you can remove pytorch110, but fix RNNT readme accordingly.

@xuechendi We need to keep pytorch1.10 for RNNT, because pytorch1.10 base image works and pytorch1.12 base image has bugs with torch_ccl

@xuechendi Could you please review and merge this PR. All comments has been fixed. DeNas related DockerfilePytorch120 works well.

@zigzagcai , hi, zheng, I found you changed backend name in start_e2eaiok_docker,py, which makes it incompatible with input argument(
https://github.com/intel/e2eAIOK/blob/fa0911113af01785c485bd3c0d48b7c5dbec78cb/scripts/start_e2eaiok_docker.py#L15
) and readme for dlrm
Can you keep backend name stays the same?

It has been fixed.

@xuechendi
start_e2eaiok_docker has been fixed, and we need to keep PyTorch1.10 for RNNT
PyTorch1.12 is dedicated for DeNas and TLK.


GEODE-8537: Solve memory increase when LRU eviction is enabled

Whenever LRU eviction was enabled it was noted an slight increase in the memory usage.
Specifically in an scenario in which a set of entries are continously created and destroyed.
Problem was that entries within LRUList where inserted but not removed until LRU
eviction happened and in the described case above, that was never.
Solution was to replace the LRUList by a refactored version called LRUQueue and
also to remove the entries from the queue upon destroy or invalidation.
Also a dead-lock between EvictionController and EvictionThread has been solved
resulting in a complete refactor.
Unit tests have been added for the LRUQueue.
Removed old integration TS for LRUList as it has now been replace by an UT TS.
Integration test have been added in the new integration test for the LRU eviction.
Also a wrongly implemented LRU eviction test was removed from the old integration tests.

Maybe @moleske @pivotal-jbarrett @mreddington @pdxcodemonkey you could throw some comments?
Thanks!

Codecov Report

Merging #687 (98a14aa) into develop (0e9463d) will decrease coverage by 0.39%.
The diff coverage is 89.42%.

@@             Coverage Diff             @@
##           develop     #687      +/-   ##
===========================================
- Coverage    74.37%   73.97%   -0.40%
===========================================
  Files          644      644
  Lines        51189    51054     -135
===========================================
- Hits         38070    37767     -303
- Misses       13119    13287     +168

Impacted Files
Coverage Δ

cppcache/integration-test/testRegionMap.cpp
100.00% <ø> (ø)

cppcache/src/EvictionController.hpp
100.00% <ø> (ø)

cppcache/src/LRUEntriesMap.hpp
76.92% <ø> (ø)

cppcache/src/LRUExpMapEntry.hpp
11.76% <ø> (ø)

cppcache/src/LRUMapEntry.hpp
80.00% <ø> (ø)

cppcache/src/LocalRegion.hpp
86.36% <ø> (ø)

cppcache/src/RegionInternal.hpp
100.00% <ø> (ø)

cppcache/test/mock/MapEntryImplMock.hpp
44.44% <44.44%> (ø)

cppcache/src/LRUEntriesMap.cpp
60.08% <63.63%> (+8.10%)
:arrow_up:

cppcache/src/LRUEntryProperties.hpp
88.88% <88.88%> (ø)

... and 32 more

Continue to review full report at Codecov.

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