When vertex/index data is updated for a mesh, the submeshes of its instances (if any) are not updated accordingly

See https://forum.babylonjs.com/t/applying-vertexdata-to-mesh-used-as-instancedmesh-source/16670/5

Not sure how to tackle that one...
The thread in the forum is using VertexData.applyToMesh, but we could update the positions/indices data by other means (mesh.setVerticesData / mesh.setIndices for eg) and the fix should work for all cases.
The easiest way would be to let the user update the instances manually by calling a method to do so. This method is already existing on InstancedMesh, it is _syncSubMeshes(). It is already public but with a leading underscore, so we would need to remove it. That would let the user take the decision to update the submeshes of the instances or not: only updating the indices on the master mesh and not on the instances could be leggit, even if it's not what the OP of the thread is needing.
The other option is to have the _syncSubMeshes() function called automatically on all instances (if any) when the positions/indices are changed, but we must make sure we update all the code paths that do that.
What do you think?

@Popov72: Just a side note from me who found this issue. In my situation the real index buffer shrunk so not updating the submeshes on the InstancedMeshes casues a crash. A solution for this problem given my limited understanding would be to have picking getting the sourceMesh subMeshes instead of the InstancedMesh-one (why does it have one anyway?). But this is just random guesses from someone who knows nothing about the design so feel free to ignore of course!

I love @breakin approach if doable @Popov72  ???

(also a sidenote: I think updating vertices work actually, this is just related to updating indices as far as I diagnosed it)

Using the master sub meshes for picking instead of the sub meshes of the instance would be a breaking change, as the sub meshes of an instance could be setup differently than the sub meshes of the master (I don't know if it happens in real scenarios, but the sub meshes being different between the instance and its master, it could definitely be). Also, that would not fix the main problem being that the indexStart / indexCount properties of the instance sub meshes having wrong values when updating the master mesh indices.

(also a sidenote: I think updating vertices work actually, this is just related to updating indices as far as I diagnosed it)

Updating the vertex positions of the master mesh must also be taken into account because a sub mesh is also defined by a vertex start / vertex count in addition to an index start / index count.

@Popov72: Hm that all makes sense. I feel ignorant here but given that submeshes on instances are ignored for rendering, why would the user change them? It would only affect picking afaict...

Thinking more about it, why not updating always all children in this case ? if you have instance of cube and you modify it to a sphere would you not expect all the instances to be impacted ?

if you have instance of cube and you modify it to a sphere would you not expect all the instances to be impacted ?

In this case I think yes, but the change can be not so drastic and you could update the indices/positions on the master mesh and use a subset of those indices/vertices in the sub meshes if you want. I don't know if it's a useful use case, though...

Yup, and in this case we should as well refresh the children cause it could lead to some pretty strange cases and other bugs otherwise.
I think we should update the applyToMesh, setVertices and setIndex to refresh the children for now and see later if we got any other needs later for the unsynced version. At Least, the current bug would be addressed in a consistent way.
Any thoughts ?

@Popov72: But for rendering it will always use the submeshes of the sourceMesh I think no matter what. I have a hard time thinking that the code that does instancing will special-handle the instances based on the submeshes (they can't be instances if they have different values)... Remember that for cloned meshes this already updates all submeshes automatically (since they are in the meshes-list in the Geometry). So there they are assumed to be automatically synched...

You're right, I have just recreated the submeshes when the positions/indices are updated, see #9660


Where to add authorization?

Hey @maxiperezc!
Thanks for the great gem! I'm really curious to give it a try in my app. :)
I'm trying to figure out where I'd add authorization, so that I can, for example, restrict access to non-published Posts, or prevent user/guest from creating new Blogs, etc.
I've found Permissions section in the docs, but (as far as I understand) it's actually only for hiding certain fields from the API.
Thanks!

@vfonic Unfortunately I haven't had time to add an authorization layer to this gem yet, but I am planning to add them as soon as I have that requirement myself.


Global CORS Configurations do not work for URLS not matching any routes

Given  A configured route matches/bootstrap URL
But, No routes matches /bootstrap2 URL.
When /bootstrap is invoked
Then API responds with 200 (As Expected)
And CORS response headers present (As Expected)
When /bootstrap2 is invoked
Then API fails with HTTP status code 404 (As Expected)
And No CORS response headers present (Failed)

See the example cURL calls and HTTP Response headers received.
curl -v 'http://<IP_ADDRESS>/bootstrap' \
  -H 'authorization: Bearer xxxx' \
  -H 'accept: */*' \
  -H 'origin: https://a.alpha.hisab.co'

< HTTP/1.1 200 OK
< transfer-encoding: chunked
< Vary: Origin
< Vary: Access-Control-Request-Method
< Vary: Access-Control-Request-Headers
< Access-Control-Allow-Origin: https://a.alpha.hisab.co
< Access-Control-Expose-Headers: Authorization, Content-Length, Content-Range
< Access-Control-Allow-Credentials: true
< X-Content-Type-Options: nosniff
< X-XSS-Protection: 1; mode=block
< Cache-Control: no-cache, no-store, max-age=0, must-revalidate
< Pragma: no-cache
< Expires: 0
< X-Frame-Options: DENY
< Content-Type: application/json
< Date: Mon, 24 Jun 2024 09:24:16 GMT
< Referrer-Policy: no-referrer

curl -v 'http://<IP_ADDRESS>/bootstrap2' \
  -H 'authorization: Bearer xxxx' \
  -H 'accept: */*' \
  -H 'origin: https://a.alpha.hisab.co'

< HTTP/1.1 404 Not Found
< Content-Type: application/json
< Content-Length: 143
< Cache-Control: no-cache, no-store, max-age=0, must-revalidate
< Pragma: no-cache
< Expires: 0
< X-Content-Type-Options: nosniff
< X-Frame-Options: DENY
< X-XSS-Protection: 1 ; mode=block
< Referrer-Policy: no-referrer

And CORS configuration is as follows:
spring:
  cloud:
    gateway:
      globalcors:
        add-to-simple-url-handler-mapping: true
        cors-configurations:
          '[/**]':
            allowed-origins:
              - https://${base-domain}
              - https://app.${base-domain}
              - https://a.${base-domain}
              - https://portal.${base-domain}
              - https://s.${base-domain}
              - https://hisab-web1.web.app
            allowed-origin-patterns:
              - http://localhost:[*]
            allowed-methods:
              - OPTIONS
              - HEAD
              - GET
              - PUT
              - POST
              - PATCH
              - DELETE
            exposed-headers:
              - Authorization
              - Content-Length
              - Content-Range
            allowed-headers:
              - '*'
            allow-credentials: true

The cors configurations can only work for matched routes.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.

The cors configurations can only work for matched routes.

Can paste a sample because using this configuration not works:
routes:
        - id: BACKEND_USER
          uri: http://localhost:8082
          predicates:
          - Path=/users/**
          metadata:
            cors:
              allowedOrigins: "*"
              allowedMethods: "*"
              allowedHeaders: "*"
              allowCredentials: true


centerVertically option

Hello,
can you please add the centerVertically option ?
https://github.com/nfroidure/svgicons2svgfont?tab=readme-ov-file#optionscentervertically
Thank you :)

Hi @sholzer  🙂
I'd happily welcome such a contribution, or I can attempt to add it myself once I get to it (unfortunately I can't give a time frame as I am pretty busy with personal matters)


DS-811 Wrapper Element

Jira
https://pegadigitalit.atlassian.net/browse/DS-811
Summary
A new wrapper element is created.
Details
A new wrapper is a basic element that contains a basic schema with content and attributes.
How to test
Pull the branch. Check the wrapper docs and confirm it's a good equivalent of the old wrapper object.
Release notes
The new e-bolt-wrapper will replace the old o-bolt-wrapper object

@danielamorse approved your suggestions


Resolves issue #447

Fixes #447, where the FreezeOnMatchCustomization conflicted with the AutoMoqCustomization – and all other auto-mocking customizations for that matter – causing the frozen mock not to be returned for requests of the implemented interface type.

Thank you for your contribution! It's now live as AutoFixture 3.36.1.


Agents not showing in Dev Portal ( UI )

Describe the bug
Agent already created using Client Code , given Id and other details like default ( that's happening even with Basic Agent ) and its showing in the backend ( sql lite, showing in the Agents table with all fields ( agent_id, user_id, name, persona , human, preset, llm config ( {"model": "gpt-4o", "model_endpoint_type": "openai", "model_endpoint": "https://api.openai.com/v1", "model_wrapper": null, "context_window": 8192} ) ,

embeddings_config ( {"embedding_endpoint_type": "hugging-face", "embedding_endpoint": "https://embeddings.memgpt.ai",
"embedding_model": "BAAI/bge-large-en-v1.5", "embedding_dim": 1024, "embedding_chunk_size": 300} ) .

Yet in the UI dev portal its not showing some of those created agents.
Please describe your setup

[ ] How did you install memgpt?
git clone, then using docker build file, then started memgpt server , then I use the password created as in the client example to create Agents
[ ] Describe your setup

Windows
How are you running memgpt? (cmd.exe/Powershell/Anaconda Shell/Terminal)
PowerShell Terminal From Windows ( using another Powershell Terminal from Visual source code to access the Server using client example )

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
def main():
# Create an admin client
token = "3Uz8TeKj4OY-qdZAWi3R5A"  # Server Token
admin = Admin(base_url="http://localhost:8283", token=token)
client = create_client(base_url="http://localhost:8283", token='3Uz8TeKj4OY-qdZAWi3R5A')

basic_agent = client.create_agent(
    name="basic_agent",
)

print(f"Created agent: {basic_agent.name} with ID {str(basic_agent.id)}")

MemGPT Config
Please attach your ~/.memgpt/config file or copy past it below.
[defaults]
preset = memgpt_chat
persona = sam_pov
human = basic
[model]
model = gpt-4
model_endpoint = https://api.openai.com/v1
model_endpoint_type = openai
context_window = 8192
[embedding]
embedding_endpoint_type = openai
embedding_endpoint = https://api.openai.com/v1
embedding_model = text-embedding-ada-002
embedding_dim = 1536
embedding_chunk_size = 300
[archival_storage]
type = chroma
path = C:\Users\king_.memgpt\chroma
[recall_storage]
type = sqlite
path = C:\Users\king_.memgpt
[metadata_storage]
type = sqlite
path = C:\Users\king_.memgpt
[version]
memgpt_version = 0.3.15
[client]
anon_clientid = 00000000-0000-0000-0000-000000000000

If you're not using OpenAI, please provide additional information on your local LLM setup:
Local LLM details
If you are trying to run MemGPT with local LLMs, please provide the following information:

[ ] The exact model you're trying to use (e.g. dolphin-2.1-mistral-7b.Q6_K.gguf)
[ ] The local LLM backend you are using (web UI? LM Studio?)
[ ] Your hardware for the local LLM backend (local computer? operating system? remote RunPod?)

When you're viewing the dev portal, are you entering the admin password or the user token ("3Uz8TeKj4OY-qdZAWi3R5A")? If you want to see agents created by the user, you should enter the user token as the dev portal password (make sure you log out first). If you log in with the admin password, you'll only see agents created with the default UUID corresponding to the admin. Sorry this isn't clear in the docs!

@sarahwooders Thanks for your response, Yes I am entering the Admin password,
I assumed the Admin should be able to view all Agents whereas the user is the one who can just see those created by him
I am also assuming if I use user Token Key created by Admin to  create Client , then using the client to create Agent , so that Agent should belong to that client , also assuming when I create using Admin pass that would be a default user assigned to client ( almost<PHONE_NUMBER>00 ) if that's what's happening or not , please verify ? )

Hi @tiro2000 - actually right now the dev portal will only show agents created by the admin user (we haven't really optimized the dev portal UI for an "admin" experience yet).
I'm adding this to the TODOs for the dev portal, thank you for bringing this up.


ZEPPELIN-1918. Fix build with Spark 2.1.0

What is this PR for?
It's my misunderstanding of SPARK_VER in travis. It is only used for downloading spark distribution. We need to specify spark profile explicitly for building with specific version of spark. This PR add new profile for spark 2.1 and fix the build issue with spark 2.1.0 because SecurityManager changes its constructor signature in spark 2.1
What type of PR is it?
[Bug Fix ]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1918

How should this be tested?
Build with spark 2.1.0 and tested it manually as below screenshot.
Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@Leemoonsoo @jongyoul @felixcheung @ejono Please help review.

wait, in travis, I don't think we want to replace 2.0.1 with 2.1.0? generally for new Spark minor version we create a new test profile for it so we would test all supported Spark versions (yes, it could get long)

LGTM. Sorry I didn't spot the version number earlier.

Tested and it works well.
Shell we add spark-2.1 profile  here and here ? \cc @AhyoungRyu
Also shell we use spark-2.1 profile in release artifact build ? \cc @minahlee

I built with "./dev/change_scala_version.sh 2.11; mvn clean package -Pscala-2.11 -Pspark-2.1" and see the same test failure that Travis saw for the new 2.1.0 build (https://travis-ci.org/apache/zeppelin/jobs/190524013):
Results :
Failed tests:
PySparkInterpreterTest.testCompletion:121 null
Tests in error:
PySparkInterpreterTest.setUp:96 » Interpreter java.util.NoSuchElementException...
Tests run: 25, Failures: 1, Errors: 1, Skipped: 0

Agree with @Leemoonsoo

@ejono I have a following commit to fix the test failure. Could you retry that ?

@Leemoonsoo  @AhyoungRyu build.md is updated, but I didn't touch create_release.sh, I will left it to @minahlee to decide.

We always have been deliver release with the most recent minor spark release, so I think it's better to update release profile to use 2.1.
@zjffdu It would be appreciated if you can update spark profile in following three lines:
https://github.com/apache/zeppelin/blob/master/dev/create_release.sh#L126
https://github.com/apache/zeppelin/blob/master/dev/create_release.sh#L127
https://github.com/apache/zeppelin/blob/master/dev/publish_release.sh#L47

@minahlee Thanks for help. I have updated these release script.

great, we are good to go then, yeah?
@minahlee btw, should we update the hadoop-2.4 in the release script - we are testing against 2.6 in CI and almost everyone out there should be running at least 2.6??

@felixcheung Let me update release script in another PR. @zjffdu do you mind to resolve conflict?

@minahlee PR is rebased.

LGTM. Merge to master and branch-0.7 if there is no more discussion

@zjffdu, I did see your follow-up commit but was already testing from that commit. I'm not sure what was wrong. Wasn't the CI (which hit the same exact error) also building from that commit? If so, it seems that there is still a problem, as CI hasn't passed yet, right?

@ejono  CI is not stable, that failure is flaky test.

@zjffdu, ok, thanks for confirming that it's just a flaky test. It seemed like it could have been related, but I did not dig into it. Have you gotten the test to pass when using Spark 2.1.0?

hi - are we good with this change? @ejono it looks like CI tests running on Spark 2.1 is passing - is that what you are asking about?

@zjffdu Do you mind to rebase again please? Let me merge once rebase is done.

@minahlee PR is rebased, CI failure is not relevant.
Failed tests:
  InterpreterOutputChangeWatcherTest.test:95 expected:<1> but was:<2>

@felixcheung, yes, I was referring to PySparkInterpreterTest.testCompletion, which I thought may have been broken in the new Spark 2.1 Travis CI profile because it was broken for that profile but not other profiles, and it was broken for me locally as well when using Spark 2.1. @zjffdu says that it is a flaky test though, so I have no concerns if that's the case.

Thanks @ejono since that is passing in Travis runs let's merge this then?
let us know if you see anything - appreciate your help.


Move pyplot to the top-level

@jasongrout this simply removes the intermediate pyplot directory as per discussion in person.

@ssunkara1 what do you think of this?

This looks good! Merging..


Reduce # of changes / make project card more streamlined

Based on whatever is decided in Network Wrangler

cc: @kulshresthaa
I think the main issue is that we want to limit the number of distinct changes so that we don't end up with something as overwhelming as the wideningMinneahahaPkwy.yml project card.
One way to do this is to:

When listing specific travel_model_ids, you won't need to specify the A and B nodes
Unless the distance is changing by more than ~1%, we shouldn't be listing them. That should make the number of changes a lot less.


feat: update icons

Automated Pull Request to update icons.

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/e9ae480151be4514/build/a610a91e-9d05-46d5-a9c4-bdf2faafd90f/artifact/8eaf701df82e25c7/p/4423b97b9398120295575af8a333bf18


Pine tree sprite change and organized tree folder structure

I have changed the pine_tree.tscn sprite to be an animated pixel tree. I have also made the tree bigger and organized the tree folder into two folders, Assets and Scenes.

This is the new texture

I think you should change the order between AnimatedSprite2D and CollisionShape2D so we can see more clearly and i think you should change the storlek of collisionshape2d like that so player can go through  the tree when it behind the tree.

I have reset the tree's position that had been changed by mistake beforehand. The shadow of the tree is now more subtle and wider. I have changed the node structure so that the collisionshape is visible when in the editor. Lastly i have changed the collisionshape from a Rectangleshape to a Capsule shape so the player won't get stuck.

It seem mergable right now for my opinion.

@Snurrigt1 @DRubnnn can you merge this?


Moving Parts React Router V4

I am looking into RRV4 and I am unsure as to where to begin. And I am just writing them down as I go through the error messages.
Currently affected files

render_context.js (changes react-router -> react-router-dom)

vulcan:routing/lib/server/router.jsx (changes react-router -> react-router-config & react-router-dom)

vulcan:routing/lib/server/router.jsx (There is no match function anymore)

Because I believe we want to keep RouterServer.run in vulcan-routing we would have to mimick match?
We could:
import { matchRoutes } from 'react-router-config'
import { matchPath } from 'react-router-dom';

routes.some(route => {
      const match = matchPath(req.path, route);

      if (match) ...do something

      return match;
});

or

const branch = matchRoutes(routes, req.url);
const promises = branch.map(({ route }) => {
  ... return or fetch data
});

I am rooting for matchRoutes, it uses matchPath .
Current routes structure:
routes =>  { path: '/',
    component:
        { [Function: GraphQL]
      displayName: 'withCurrentUser(App)',
      WrappedComponent:
       { [Function: App]
        propTypes: [Object],
        childContextTypes: [Object],
         displayName: 'App' },
      contextTypes: { client: [Object], getQueryRecycler: [Object] } },
   indexRoute:
    { name: 'movies',
     componentName: 'MoviesList',
      component:
       { [Function: GraphQL]
         displayName: 'withCurrentUser(withApollo(withState(withList(MoviesList))))',
         WrappedComponent: [Object],
         contextTypes: [Object] } },
   childRoutes:
    [ { name: 'resetPassword',
       path: '/reset-password/:token',
        componentName: 'AccountsResetPassword',
        component: [Object] },
     { name: 'enrollAccount',
       path: '/enroll-account/:token',
       componentName: 'AccountsEnrollAccount',
        component: [Object] },
      { name: 'app.notfound',
        path: '*',
        componentName: 'Error404',
        component: [Object] } ] }

The structure we might need:
const routes = [
  {
    component: App,
    routes: [ { name: 'resetPassword',
       path: '/reset-password/:token',
        componentName: 'AccountsResetPassword',
        component: [Object] },
     { name: 'enrollAccount',
       path: '/enroll-account/:token',
       componentName: 'AccountsEnrollAccount',
        component: [Object] },
      { name: 'app.notfound',
        path: '*',
        componentName: 'Error404',
        component: [Object] } ]
  }
];

Sorry, this isn't in order but rather a reflection of where I am with RRv4. What are your thoughts?

I think it would be wise to allow a swap between 3 or 4. I’m not sure everyone would be happy to have a wholesale swap to 4. Would be a large breaking change.

https://github.com/ReactTraining/react-router/blob/master/packages/react-router/docs/guides/migrating.md

I like your idea @justinr1234. You could swap packages but the way you add the routes should stay the same, correct? What bugs me about the migration guide is that someone one Medium wrote a better one (look for unofficial migration guide react router 4). Most of the posts do not talk about SSR aspects @SachaG . Also, I went through some of the starter apps (express and RR4) and it is really inconclusive which one seems to work better. One that I like is here I keep looking.

I think it would be wise to allow a swap between 3 or 4. I’m not sure everyone would be happy to have a wholesale swap to 4. Would be a large breaking change.

Not sure I agree. Seems like supporting both at the same time or even just swapping would be a pain to maintain, and also potentially cause a lot of confusion.

Also true.

It's still on the roadmap… that being said Vulcan has its own thin wrapper on top of RR so ideally it shouldn't matter which version is actually used.

RR4 is used in the new Apollo 2 version. Breaking changes are numerous but most are documented in the React Router update doc, Vulcan's API is mostly the same.


New package: MultiKDE v0.1.0

Registering package: MultiKDE
Repository: https://github.com/noil-reed/MultiKDE.jl
Created by: @noil-reed
Version: v0.1.0
Commit: cbdbbbaa431aa9456fcf27978a2544d8c0c15822
Reviewed by: @noil-reed
Reference: https://github.com/noil-reed/MultiKDE.jl/commit/cbdbbbaa431aa9456fcf27978a2544d8c0c15822#commitcomment-52734674

stop merging


[BUG] Unable to save new variable

Describe the bug
Not able to save a new variable
GSE Version

Version: [e.g. 3.0.0-Beta-RC2]

Describe the bug
Still not able so save variables (classic tbc).
Sometimes its possible to save but the variable name will be removed and sooner or later the whole tab is gone again.
GSE Version
3.0.0-Beta-RC2a


minor improvements in sriovOpeatorConfig controller

move log level handling earlier in reconcile flow
add log msg when reconicle has completed successfully

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 25.838%

Files with Coverage Reduction
New Missed Lines
%

api/v1/helper.go
3
42.68%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
2905

Relevant Lines:
11243

💛  - Coveralls


How to build?

I'm new to Chrome extensions but would like to play with this one some.  Please add instructions on how to build and install.

Simply click "Load Extension" in Chrome and point to the code


updateDictionaries() always throws error if dictionary for the locale is not available

updateDictionaries() is using user's locale, so that hunspell try to load a dictionary for the locale. If dictionaries for the locale is not available (like in "ja_JP" locale), hunspell exits with error claiming "Can't open affix or dictionary files for dictionary named "ja_JP"." Thus, the fucntion always popups "Call to hunspell failed with a code of 1" error, which is irritating.
How about calling the process with LANG=C instead? It's not necessary here to honor user's locale.

I encounter the same issue with hunspell.
It would be good to add -d {defaultLangages} when executing updateDictionaris().
@naota
I found an ad-hoc solution.
You can avoid the error message by adding two empty files named ja_JP.dic and ja_JP.aff into ~/Library/Spelling (if you use macOS) or somewhere else in the path of hunspell.


Make storage client work correctly with read only secondary accounts

We need to remove the "-secondary" string from the account name when using it to authorize access.
See also here: https://blogs.technet.microsoft.com/ksasaki/2013/12/23/use-ra-grs/.

See also here: https://blogs.technet.microsoft.com/ksasaki/2013/12/23/use-ra-grs/.

@dovreshef do you know if there is an English version of this article?

@ahmetalpbalkan that's the only mention of the issue that I found. Had to use Google translate :).

@dovreshef  I'm trying to understand why would you specify storage account name as the -secondary endpoint in your code. Why do you need to hit the secondary endpoint instead of the primary one?

@ahmetalpbalkan We use it to simulate a regional downtime, read all the backups off the storage.

Ah okay, in that case, merging.

@ahmetalpbalkan thanks!!


Client displays crash message instead of kick message when kicked for Forge mod

Love this plugin, but I'm having an issue.
Latest version 4.0.4 on hosted Spigot 1.10.2. Not many plugins but I will reproduce with just this one and it's dependencies if needed. No errors on load.
After installing, enabling Forge protection, and restarting the server (a couple of times), when I join with forge-1.10.2-<IP_ADDRESS>7 (which reports as beta on launch) I am kicked as expected, but instead of the expected message to the player (punish-command: kick %player% The Forge mod is not allowed.) the client crashes with a normal crash error "Connection Lost - Internal exception blah blah blah connection forcibly closed by the remote host.". Notice I took out &4 as part of my testing. I'll download some of the other mods to see if the behavior is global.
I don't want to make anyone think there's a problem with their client. I'll test some more with different versions of Forge. Thank you!

Oops. I forgot to mention the logs look great. I'll get them from my laptop and paste the link here.

Hello, the kick message isn't 100% probable to appear. I still don't know the reason of this, but it's the same for the world downloader mod and the liteloader mod.

Gotcha. Thank you for the response. Again, a very awesome mod!


Unable to Find execute this yaml, error says 'Kind' is missing

Hi during deployment of this yaml code, it says following error all of sudden though it used to work previously.
error: unable to decode "deptest.yaml": Object 'Kind' is missing in '{"resources":[{"name":"quickstart-deployment-vm","properties":{"disks":[{"autoDelete":true,"boot"
Please suggest the corrective action, i need to take.

This is resolved by myself. Actually, i was trying to deploy yaml using kubectl instead of using deployment-manager. I actually overlapped while working in GKE vs non GKE environment.  ;-)

Given that you are working across mixed environments you may want to look
at https://github.com/GoogleCloudPlatform/k8s-config-connector
On Wed, Jun 26, 2019, 10:53 PM pankajjeet001<EMAIL_ADDRESS>wrote:

This is resolved by myself. Actually, i was trying to deploy yaml using
kubectl instead of using deployment-manager. I actually overlapped while
working in GKE vs non GKE environment. ;-)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/deploymentmanager-samples/issues/472?email_source=notifications&email_token=ALZVKCLK3Q7TOYQHS3TBT7TP4RIW3A5CNFSM4H3YCBB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYWAMOY#issuecomment-506201659,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALZVKCLK66H4TTAIRE7EAI3P4RIW3ANCNFSM4H3YCBBQ
.


Add missing requiresment to requirements-dev

Sendfile is required for running the tests, it's present in the tox.ini but not for normal development.

Cheers! Have merged this now so that it isn't getting in the way of development, but ideally we should fix the test setup so that those tests are skipped if sendfile isn't available (as we do for Embedly and ElasticSearch) - see #1367.

Couldn't agree more, but the initial issue is that the tests simply won't run as it's listed in INSTALLED_APPS!


Duplicated styles

Islands sometimes result in hydrated and non-hydrated styles containing overlapping duplicates. The bug isn't severe, just annoying and inefficient.
I'm not sure how to fix it, so someone else will probably need to resolve it. Hopefully the official plugin gets updated soon.

This turned out to be as simple as using a different method for getting Twind styles. stringify, inline, extract, and convert all do really similar things and the comments don't really make it clear which one is which.
I was using extract when I should've been using stringify.


Patch to fix #209

Awaiting confirmation that 3000 chars is ok - this is for issue #209

Current coverage is 22.98%

Merging #213 into master will not affect coverage as of 18c0357

@@            master    #213   diff @@
======================================
  Files           10      10
  Stmts          596     596
  Branches         0       0
  Methods         80      80
======================================
  Hit            137     137
  Partial          0       0
  Missed         459     459

Review entire Coverage Diff as of 18c0357

Powered by Codecov. Updated on successful CI builds.


Add wait to flush interceptors #2397

Adding a small wait before closing browser to handle #2397
Signed-off-by: Zabil Cheriya Maliackal<EMAIL_ADDRESS>

Also notice this fix seems to have fixed the flakiness in tests which was failing earlier at after Scenario closeBrowser


Rowset File updated

https://github.com/microsoft/SqlNexus/issues/66
Required update to fix the above issue has been done in TextRowset.xml file , please review the same and merged.

@prmadhes-msft , could you pull the latest on this branch and test it to make sure this fixes the issue

Tested together with @prmadhes-msft . Merging into Master


mod-admin-full: Allow to create multiple DHCP/DNS server instances

Trunk LEDE now has ability to launch multiple instances of
DNSMasq which can be useful.
Signed-off-by: Daniel Dickinson<EMAIL_ADDRESS>

@jow- @hnyman @chris5560 Any objections?
@EricLuehrsen Do you notice any problems with this for what you've been doing?

@cshoredaniel: it shouldn't be an issue with what I am trying to do, although something in dnsmasq-dhcpv6 UCI is upsetting a few folks, so now I need to recheck what I was trying to do. I would let @jow- speak to this more, but dnsmasq instances can be a bit unusual for the novice user, so we should be careful how we present it in LuCI.

Closing as have taken a break from LEDE/OpenWrt development for various reasons, so will reopen when this is back on the radar.


Sftp LIBSSH2_ERROR_* caused panic during Drop

Hello,
I am facing an issue with Sftp subsystem where if any protocol error happens during Drop, it causes a panic which is not catchable, resulting in undefined errors.
Example:
thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `-7`,
 right: `0`', /home/bigo/.cargo/.../ssh2-0.8.0/src/sftp.rs:645:13

This specific example is trying to return LIBSSH2_ERROR_SOCKET_SEND
Is there any reason why this piece of code does not return an Error instead of just panicing?
Further more, is there any way for me to catch this panic? I have tried unwinding the stack, but I can't make it work..
The panic can come from these 2 assertions inside SFTP:

https://github.com/alexcrichton/ssh2-rs/blob/master/src/sftp.rs#L435
https://github.com/alexcrichton/ssh2-rs/blob/master/src/sftp.rs#L645

Is there any reason why this piece of code does not return an Error instead of just panicing?

We can't change the signature of the Drop trait :-/
@spebern: what do you think about removing the asserts from these locations?  I don't think there is any sane error recovery that can be done, so it is probably fine to ignore this error.

Silly me by asking about returning an error.. you cannot change this trait signature, of course..
Regarding the safety of ignoring this error, I think it is fine.. obviously something went catastrophically wrong, but we are very late in the execution here and there is nothing you can really do.
It would be nice to have some sort of notification thou that something went really wrong..

@wez I don't think those asserts should be necessary

Allright, I'll work on a PR today.

This was published as part of ssh2 0.8.1

Thanks a lot @wez!


Avoid Guice circular issue and implement test to ensure extension is activated

Testing done
Automated tests
### Submitter checklist
- [x] Make sure you are opening from a **topic/feature/bugfix branch** (right side) and not your main branch!
- [x] Ensure that the pull request title represents the desired changelog entry
- [x] Please describe what you did
- [ ] Link to relevant issues in GitHub or Jira
- [ ] Link to relevant pull requests, esp. upstream and downstream changes
- [ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Can you take a look @sridamul

Can you also remove the validation? or need to figure out other way to display the message correctly.

Can we use the extension from the extension Indexer? If possible, we can use it after the completion of this GSoC'24 project

Can you also remove the validation? or need to figure out other way to display the message correctly.

https://github.com/jenkinsci/extension-filter-plugin/pull/70
By reading doc this can be achieved using an other class loader to check class existence

Can we use the extension from the extension Indexer to validate class? If possible, we can use it after the completion of this GSoC'24 project

Not sure what you mean. But we are talking about 2 different tool here (One plugin that can disable extension on a running controller and one tool that just display the available extension across all plugin)

Not sure what you mean. But we are talking about 2 different tool here (One plugin that can disable extension on a running controller and one tool that just display the available extension across all plugin)

I thought that one can check whether the class names are available in the extension indexer. If it presents, then it is an extension and can be indicated with correct validation
As you suggested

By reading doc this can be achieved using an other class loader to check class existence

this is better approach.

Can you take a look @sridamul

tested the code locally (both descriptor and extension)
LGTM!

If possible can you remove the default="&{null}"in the https://github.com/jenkinsci/extension-filter-plugin/blob/main/src/main/resources/org/jenkinsci/plugins/ConfigurableExtensionFilter/global.jelly#L12
as suggestion by 'Daniel Beck'


Cannot hook to i386 apps after upgrade to 10.12

I've created a plugin to hook into a 32bit only app working on SIMBL, since the app is written with i386 so I can only run the plugin in i386 mode, after upgrade to sierra and updated to the latest SIMBL, I found seems like it starts to failed to load i386 plugins and start to alert me like this, how can I work with it?
Error loading /System/Library/ScriptingAdditions/SIMBL.osax/Contents/MacOS/SIMBL:  dlopen(/System/Library/ScriptingAdditions/SIMBL.osax/Contents/MacOS/SIMBL, 262): no suitable image found.  Did find:
/System/Library/ScriptingAdditions/SIMBL.osax/Contents/MacOS/SIMBL: mach-o, but wrong architecture
32BIT_APP: OpenScripting.framework - can't find entry point InjectEventHandler in scripting addition /System/Library/ScriptingAdditions/SIMBL.osax.

I managed to recompile SIMBL framework with Universal and it works as normal

@eminarcissus how did you do it? I had a compilation error when I changed architecture.


[GFIconButton] Wrong touch area

Describe the bug
GFIconButton does not detect tap event properly.
To Reproduce
Too long story and please check below screen record.
https://www.dropbox.com/s/my4gbswuwy1sfv7/Screen Recording 2020-10-08 at 13.54.29.mov?dl=0
Expected behavior
GFIconButton should be clickable for all splash area.
Smartphone (please complete the following information):

Simulator : Pixel-XL(android 31), iPhone X (iOS 14)
Real device : Samsung Galaxy, iPhone X.

This issue has been fixed in new release 1.2.3


Use requests.Session while communicating with Marathon.

Right now when Client opens new tcp connection for any call to Marathon api. Use requests.Session provide great performance improvement by ensuring keep-alive connections when many sequential calls are made.

Oooo


RpcError: -32000 invalid sender'

Error seems to suggest that the wrong chain config is being used.
Confirm we're testing against a rinkeby eth node
$ curl -X POST -H "Content-Type: application/json" --data '{"jsonrpc":"2.0","method":"net_version","params":[],"id":67}' http://<IP_ADDRESS>:8080/ethereumrpc/
{"jsonrpc":"2.0","id":67,"result":"4"}

Geth log error
WARN [12-05|15:09:16.534] Served eth_sendRawTransaction            conn=<IP_ADDRESS>:42954 reqid=1575558555996 t=1.272341ms err="invalid sender"

Agent Config:
  [assets.ETH]
  network = "rinkeby"
    [assets.ETH.rpc]
    url = "http://<IP_ADDRESS>:8080/ethereumrpc/"

Error:
2019-12-05T14:35:26.947Z liquality:agent:worker Max attempts reached. Job has failed { _id: 5de91557e86c4452558496ef,
  name: 'reciprocate-init-swap',
  data: { orderId: '7be6c5a3-41f6-4c83-9874-0353e008d8e5' },
  type: 'normal',
  priority: 0,
  nextRunAt: null,
  lastModifiedBy: null,
  lockedAt: null,
  lastRunAt: 2019-12-05T14:35:25.595Z,
  failCount: 6,
  failReason:
   'http://<IP_ADDRESS>:8080/ethereumrpc/ - RpcError: -32000 invalid sender',
  failedAt: 2019-12-05T14:35:26.945Z,
  lastFinishedAt: 2019-12-05T14:35:26.945Z }

https://github.com/liquality/chainabstractionlayer/commit/f180faa54e0dd4d05a284a0497067e1edf142aa1
Fixes this

Confirmed in testing. Thanks


Regression in 3.2: Invalid charater

[ERROR] SQL State  : 22019
[ERROR] Error Code : 911
[ERROR] Message    : ORA-00911: invalid character
[ERROR] Location   : ddl/db/migration/V2__initial_schema.sql (xxxx)
[ERROR] Line       : 1
[ERROR] Statement  : CREATE TABLE FND_ACCOUNTS (
I've configured
${project.build.sourceEncoding} to UTF-8 - and the file is UTF-8.
It runs fine with flyway 3.1 - same drivers etc.

Thanks for the report. Please post the full offending statement, so we can include it in our test cases.
Cheers
Axel

I can canfirm an Oracle 00911 bug. Take: "COMMENT ON COLUMN SATZ_MARKE.KURZZEICHEN    IS 'Kurzzeichen';" <-- Important that it is a single word comment
Then: org.flywaydb.core.internal.dbsupport.SqlStatementBuilder.addLine(String)
does not detect the "terminated" property because org.flywaydb.core.internal.dbsupport.SqlStatementBuilder.endWithOpenMultilineStringLiteral() says true and therefore terminated is NOT set to true although it should. Flyway then continues thinking that three statements are one.
line=COMMENT ON COLUMN SATZ_MARKE.KURZZEICHEN IS 'KURZZEICHEN';
delimitingTokens=[OTHER, OTHER, OTHER, OTHER, OTHER, QUOTE]
Bug seems to me in:
org.flywaydb.core.internal.dbsupport.oracle.OracleSqlStatementBuilder.cleanToken(String)
(I expect QUOTE twice)
Script part:
COMMENT ON COLUMN SATZ_MARKE.KONZERN_ID     IS 'Referenz auf einen Konzern';
COMMENT ON COLUMN SATZ_MARKE.KURZZEICHEN    IS 'Kurzzeichen';
COMMENT ON COLUMN SATZ_MARKE.STRUKTUR_ID    IS 'Referenz zur Struktur';
CREATE OR REPLACE TRIGGER SATZ_MARKE_TR1 BEFORE DELETE OR INSERT OR UPDATE ON SATZ_MARKE
...

@sho81 Thanks for the analysis! This will be hugely helpful in fixing this.

Hi,
I also got this Oracle 00911 error and I agree in the analysis of @sho81.  It seems like the reason that "org.flywaydb.core.internal.dbsupport.SqlStatementBuilder.endWithOpenMultilineStringLiteral() says true and therefore terminated is NOT set to true although it should" is because of the regexp-string in cleanToken-method.
I got this error in an insert with an single quoted strings that ended with an N. When I debug the cleanToken-method I see 'myStringEndingWithA_N'-token match the regexp-string and return just a single-quote.
I wonder if this pull request from @soemeier is the same error: https://github.com/flyway/flyway/pull/973
The code-change in the pull-request, fix my "Illegal-character"-problem. It seems like the Oracle error was introduced iin  699571a6bc15
Cheers
Christine Teig

Hi,
I agree. We probably have the same issue. So my bug would be a duplicate.
Best Regards
Soeren

It's definitely the 'N'. I have to correct my first assumption that it must be a single word comment. Flyway 3.1 works fine. I assume there will be a hotfix to this in 3.2.(0.)1 - eagerly awaiting...


fix(emergency_handler): stop using heartbeat_checher to make subscriber unique

Description
Now, subscriber for hazard_status is duplicate due to processing data and checking heartbeat.
I think a subscriber should be as unique as possible for a topic.
I'll fix it.
Moreover, I'll delete heartbeat checker implementation because it is only used in emergency_handler
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

I'ts true, but deleting and integrating heart beat library is more simple because the library is only used at only one node now.

FYI https://github.com/autowarefoundation/autoware.universe/pull/1501#issuecomment-1207698595


Fix missing WindowOrWorkerGlobalScope partial interface mixins.

Without the mixin keyword, wasm_bindgen_webidl would report:
Partial interface WindowOrWorkerGlobalScope missing non-partial interface

Also, including the "mixin" keyword here is consistent with the official
webidl spec (for example see https://fetch.spec.whatwg.org/#fetch-method).
The most important thing this change unlocks is the fetch method (CC #449)

With this change, the only remaining instance of "missing non-partial interface" is:
Partial interface ConsoleInstance missing non-partial interface

Which I thing comes from the following (which looks ignoreable):
// this interface is just for testing
partial interface ConsoleInstance {
  [ChromeOnly]
  void reportForServiceWorkerScope(DOMString scope, DOMString message,
                                   DOMString filename, unsigned long lineNumber,
                                   unsigned long columnNumber,
                                   ConsoleLevel level);
};

Looks like a great fix to me, thanks for tracking this down!


process type failure

I tried building this on a spring boot and now get this when running pack build:
ERROR: failed to build: Near line 2 (last key parsed 'processes.type'): expected value but found "web" instead

It was my local project.toml. Oops!


MongoDbSagaRepository: can't define custom mongo collection

In version 3.3.5 (currently - latest stable NuGet package version) there no way to define custom mongo collection in MongoDbSagaRepository istance or change _collection in derived class.

I left the default, but made it possible to override the collection name now.


Merge from RAG-on-GKE PR#3

Make embdding restful
Support text only embedding
Fix return type : now returns vector instead of tensor

Can you switch this to the staging branch? We'll stage all of the RAG changes there until we are ready to release them to main.

Can you add the copyright headers to your files.

I'm going to close this PR as I believe development has moved to another branch.


Adds date and code block instructions to AssistantAgent prompt.

[Draft]
Why are these changes needed?
There are several opportunities to improve the default AssistantAgent system message to:

Make the agent aware of the current date. (#428)
Provide more explicit instructions for formatting code (more important for small LLMs)

Related issue number
#428
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (d22664f) 27.89% compared to head (12c5856) 37.49%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #738      +/-   ##
==========================================
+ Coverage   27.89%   37.49%   +9.59%
==========================================
  Files          27       27
  Lines        3466     3467       +1
  Branches      784      784
==========================================
+ Hits          967     1300     +333
+ Misses       2428     2047     -381
- Partials       71      120      +49

Flag
Coverage Δ

unittests
37.43% <100.00%> (+9.59%)
:arrow_up:

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

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

Initial testing with a reduced 26-problem version of HumanEval shows promise. No appreciable difference for GPT-4 (green) or GPT-3.5-turbo-16k (red), but sizeable boosts to older GPT3.5-turbo-0301 (blue) and Zephyr-7B (purple).
Gains mainly come from adherence to Markdown code block formatting, aiding detection and execution by the UserProxyAgent.

@afourney shall we revisit this post results with GAIA?

@gagb I don't expect it to impact GAIA much because GAIA basically requires GPT-4 (it's too hard for other models), and GPT-4 is ok with this.
This change is mostly for the benefit of weaker models like GPT-3.5

Please reopen if you want these changes.


Generated the xlf files for the CancellationToken changes

Generated the xlf files for the CancellationToken changes

Loc team has recently moved to a new tool and these things are happening after that. They said it would be a one time thing.


Is it possible to add custom attrs from fiber context?

For example request/response body or request ID.
Currently there is request id being added (that can be disabled).
What if I want to use the request ID middleware, in order to use a custom header / generator.
How I could log that?
Thanks!

I suppose we can do something like that:
slogfiber.NewWithConfig(
    logger,
    slogfiber.Config{
        CustomAttrs: func (ctx *fiber.Ctx) []slog.Attr {
              .......
        },
        WithRequestBody: bool,
        WithResponseBody: bool,
    },
)

WDYT ?

Looks good yea

Added to v1.5.0 (see readme).


Search API

Reverts standard/standard#1962

We're still using standardrb.
I'm unsure why someone anonymous would like to remove SearchApi from the users list. :)

Closing based on previous comment


How to implement Dual encoder in fairseq as described in Learning Cross-Lingual Sentence Representations via a Multi-task Dual-Encoder Model

I am trying to use fairseq for implementing the paper titled Learning Cross-Lingual Sentence Representations via a Multi-task Dual-Encoder Model.
The encoder and decoder for the translation task share the same weights.
But I am having difficulties in modelling the dual encoder setup .
Any pointers in this direction would be really helpful.

fairseq Version (e.g., 1.0 or master):master
PyTorch Version (e.g., 1.0) 1.3.1
OS (e.g., Linux): Ubuntu 18
How you installed fairseq (pip, source): pip
Build command you used (if compiling from source):
Python version: 3.7
CUDA/cuDNN version: 10.1
GPU models and configuration: Nvidia v100 x4
Any other relevant information:

Can you elaborate more on what specifically you are trying to accomplish?  What have you tried so far?

Closing due to inactivity.


Mvt's not being removed from Map

Mv'ts are not being removed from the map after zooming out past their zoomlevel.

Fixed by adding source.clear to the return from current table check in the mvt tileUrlFunction function.


NACS Read Only Access

What: A feature request from LAN for read only access on the NACS portal.
User Story:
As a NACS administrator I want the ability for the NACS portal to have both edit and view only functionality
Value / Purpose
This feature allows people to be granted access to view the configuration within the portal without the risk of them unintentionally impacting the live service
Useful Contacts

Andy Banks supposedly has view only access. It is worth investigating this.

Confirmed that the current NAC application does not have a "read only" user role and therefore this ticket is required to create this new user role.


Document what's new in ASP.NET Core for .NET 9 Preview 2

Update the "What's new in ASP.NET Core 9.0" for .NET 9 Preview 2
@adityamandaleeka @mkArtakMSFT Please have folks comment on this issue with the desired content.
FYI: @samsp-msft @JeremyLikness @mikekistler @claudiaregio @JamesNK @SteveSandersonMS @davidfowl

Associated WorkItem - 212924

@javiercn can you please add a comment here about content that should go with the Constructor Injection work that you brought back? Thanks!

@javiercn and another one for the Blazorpack compression please.

Constructor injection: It is now possible to inject dependencies via the constructor of a component instead of using @inject.
Blazor Web Server Components compression: Compression is now enabled by default for interactive server components. This comes with a restriction to framing the app within the same origin by default, which is achieved by a content-security policy. The compression settings as well as the framing behavior can be adjusted by passing a callback to AddInteractiveServerRenderMode

I've also added primary ctors.
I've placed the guidance now (today, 3/1), and I note in the section headings "(Preview 2, mid-March)". I'll adjust that when Pre2 lands.
I see that multithreading has been pushing to Pre3. Let me know if there's anything else outside of compression and ctors. 👂
https://learn.microsoft.com/aspnet/core/release-notes/aspnetcore-9.0

I see that multithreading has been pushing to Pre3. Let me know if there's anything else outside of compression and ctors. 👂

@guardrex We had discussed potentially pushing out announcing the multithreading support to Preview 3, but after discussing it this week with the WebAssembly runtime folks (@lewing @pavelsavara) the plan is to move forward with announcing it for Preview 2. Some details on what’s supported can be found here: https://github.com/dotnet/runtime/blob/main/src/mono/browser/runtime/multi-threading.md.

Thanks, I'll make a note of that and see about placing it (commented out) sometime next week.

Thanks, I'll make a note of that and see about placing it (commented out) sometime next week.

@guardrex See updated multithreading status above

OIDC and OAuth Parameter Customization
The OAuth and OIDC authentication handlers now have a new AdditionalAuthorizationParameters option to make it easy to customize authorization message parameters that are usually included as part of the redirect query string. Previously this would have required a custom OnRedirectToIdentityProvider callback or overridden BuildChallengeUrl method in a custom hander. For example:
builder.Services.AddAuthentication().AddOpenIdConnect(options =>
{
    options.Events.OnRedirectToIdentityProvider = context =>
    {
        context.ProtocolMessage.SetParameter("prompt", "login");
        context.ProtocolMessage.SetParameter("audience", "https://api.example.com");
        return Task.CompletedTask;
    };
});

Now becomes:
builder.Services.AddAuthentication().AddOpenIdConnect(options =>
{
    options.AdditionalAuthorizationParameters.Add("prompt", "login");
    options.AdditionalAuthorizationParameters.Add("audience", "https://api.example.com");
});

danroth27 marked this conversation as resolved.
Thank you @joegoldman2 for this contribution!

Configure HTTP.sys extended authentication flags
You can now configure the HTTP_AUTH_EX_FLAG_ENABLE_KERBEROS_CREDENTIAL_CACHING and HTTP_AUTH_EX_FLAG_CAPTURE_CREDENTIAL HTTP.sys flags using the new EnableKerberosCredentialCaching and CaptureCredentials properties on the HTTP.sys AuthenticationManager to optimize how Windows authentication is handled. For example:
webBuilder.UseHttpSys(options =>
{
    options.Authentication.Schemes = AuthenticationSchemes.Negotiate;
    options.Authentication.EnableKerberosCredentialCaching = true;
    options.Authentication.CaptureCredentials = true;
});

Thank you @evgenykotkov for this contribution!


[JS] npm packages need to include a LICENSE file in the root

lerna WARN ENOLICENSE Packages adaptivecards-controls, adaptivecards-designer, adaptivecards-fabric, and adaptivecards-templating are missing a license.
lerna WARN ENOLICENSE One way to fix this is to add a LICENSE.md file to the root of this repository.

sdsdfdsf

sdsdfdsf

Hello FUSE labs

Hello FUSE labs

Hello FUSE Labs

Hello FUSE Labs

Hello demo

Hello demo

@matthidinger is this a legit issue?

@matthidinger is this a legit issue?

n/m this was demo

n/m this was demo

Test

Test

closing this again 🎉

this is a demo for bill

Test


fix: FE api service get user preset endpoint

Codecov Report

Merging #84 (e0f704b) into main (dce14a1) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #84   +/-   ##
=======================================
  Coverage   98.56%   98.56%
=======================================
  Files          61       61
  Lines         904      904
  Branches      173      173
=======================================
  Hits          891      891
  Misses         13       13

Impacted Files
Coverage Δ

client/src/utils/ApiService.ts
96.38% <100.00%> (ø)

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 dce14a1...e0f704b. Read the comment docs.


[Question] How to deal with cold start

Hi mabwiser team,
Thank you for this great library!
I am wondering what would be the best approach for mabwiser to be used in cold start situations, where there aren't yet any recorded decisions and rewards. Could it be possible to initialize rewards for all arms with 0? The use case I have in mind has binary rewards [0,1].
Thanks in advance!

Hi @risufaj!
Thank for your interest in the library.
Great question! Cold-start is a challenging problem in many recommender system problems. Note we have a warm_start function in the MAB object that can be used to warm-start cold arms. In order to warm-start the multi-armed bandit we need a list of features for each arm that can be used to calculate pairwise similarities between the arms. See here for an example how it's used.

Hi @bkleyn,
Thank you for answering!
I understand that using the function you mentioned, it would be possible to warm start an arm that is most similar to some existing arms, for which you have data. In situations where there aren't any data recorded for any of the arms, I suppose the best could be to have some sort of data collection phase, right?

Yes, indeed! This is in fact an active research area, and we recently studied how to speed-up that data collection phase with publications at CPAIOR'21 and IJCAI'21.


switch post-hardfork addresses to bech32 encoding

because they are nicer and let us put P9 in front of them (T9 for testnet)

This is primarily a matter of adding to the encode/decode functions to recognise and validate both types of address encodings from string format.


-hack-undef should only replace numeric types

We need to allow pointer-to-image use an Undef value.
This occurs when we pass an image into a helper function
but the helper function ignores the image value.  The optimizer
will replace the call parameter with an Undef pointer-to-image.
That must remain.

Tested on Release and Debug builds.
Avoids downstream issues with the optimizer.
LGTM.

Rebased and pushed onto master as  c66b335


Room list doesn't refresh

Hi @DmitrySharabin
Very nice app! congratulations !
I connected from 2 different accounts (to talk to myself... yep, at least I sometimes agree with myself  :) )
and noticed that when user 1 creates a new room, the room list doesn't refresh on other connected user 2
(even if I logout & login again  in fact !
I'm connected from same computer, with 2 different browsers:
1 is connected with Google account,
the other browser is connected with GitHub)
Even if user2, creates another Room, he doesn't see previous room created by User1 on the list (and vice versa)

-> maybe a bug?
(or is this a limitation of mavo to have same default strorage and don't refresh?)

I looked at your app, because I'm trying to solve my need of editing "fake properties" from multiple places :)
So I also created this discussion, in case you could explain how you're doing it with your app

It looks like Room 1 hasn't been created. Hm. Interesting.


Give fleet adapters the option to publish fleet states (on by default)

This has been tested in simulation and on hardware, and appears to be working as intended.


Re-designing of club cards

We propose adopting a card-based design similar to Airbnb's layout to display clubs on our website. This design is visually appealing and user-friendly, allowing for an engaging presentation of clubs while keeping the content organized and easily scannable.
Proposed Functionality:
Each club will be displayed in a card format, with the following key elements:
Club image as the header of the card.
Club name, brief description, and key details (e.g., meeting times or upcomi

ng events) below the image.
A clear "Join" or "Learn More" button to encourage interaction.
The cards will be responsive and optimized for both desktop and mobile screens, ensuring a seamless user experience across devices.
The layout will be designed to support multiple cards in a grid format, with appropriate spacing and padding to maintain visual clarity.

We propose adopting a card-based design similar to Airbnb's layout to display clubs on our website. This design is visually appealing and user-friendly, allowing for an engaging presentation of clubs while keeping the content organized and easily scannable. Proposed Functionality: Each club will be displayed in a card format, with the following key elements: Club image as the header of the card. Club name, brief description, and key details (e.g., meeting times or upcomi  ng events) below the image. A clear "Join" or "Learn More" button to encourage interaction. The cards will be responsive and optimized for both desktop and mobile screens, ensuring a seamless user experience across devices. The layout will be designed to support multiple cards in a grid format, with appropriate spacing and padding to maintain visual clarity.

Sorry but the current cards match the minimilistic look and currently the image uploading of clubs aren't working so we'll look it afterwards.


Dependency updates

Description

Related issue: #ISSUE_NUMBER
Motivation and Context

How has this been tested?

Screenshots (if appropriate)
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Checklist

[ ] I have read the contributor guide.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

160 of 193 (82.9%)  changed or added relevant lines in 24 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.6%) to 79.046%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

rdmo/projects/permissions.py
6
7
85.71%

rdmo/projects/utils.py
0
1
0.0%

rdmo/projects/validators.py
16
18
88.89%

rdmo/services/providers.py
0
4
0.0%

rdmo/projects/viewsets.py
14
24
58.33%

rdmo/projects/templatetags/projects_tags.py
7
22
31.82%

Files with Coverage Reduction
New Missed Lines
%

rdmo/projects/viewsets.py
2
65.96%

Totals

Change from base Build<PHONE_NUMBER>:
0.6%

Covered Lines:
7131

Relevant Lines:
8775

💛  - Coveralls


Error while pulling docker image.

Hello ,
Getting same error all the time whenever trying to pull docker image with command
docker pull caioquirino/docker-cloudera-quickstart
System : Ubuntu 14.04 LTS
Error is :
FATA[333462] Get  https://index.docker.io/v1/repositories/cloudera/quickstart/images: dial tcp: lookup index.docker.io: no such host
Please provide a solution.

Hi @pswapnil1 ,
Please review this issue, the error can be related with your machine's connectivity.
Please try to use docker pull mysql:latest for a simple test.


Fix incorrect envvar name in gpg error

Purpose / Changes
An error message in website/security.py incorrectly referred to the GNUPG_BINARY settings key as GNUPGBINARY (missing underscore)
Side effects
None expected.
Ticket
No ticket

This is obviated by #6391 , which should be shipped during the next release.


Make whole TestECSSuite e2e test as flaky

What does this PR do?
Mark the whole TestECSSuite as flaky.
Motivation
Snooze impact on CI and other branches while investigating failures in the suite during #incident-28805.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


create Maybe from value

Hello,
I would like to know if there is a way to create Some or Nothing based on a value. Specifically, if the value is None, it should create Nothing, and if the value is present, it should be encapsulated within Some.
I saw a great example of this in one of the libraries that was put as an alternative to your repositories. This implementation is the one I am referring.
If this functionality does not currently exist, I am willing to implement it if we can agree on the approach.

Hi,
I think a Maybe.from makes sense. But we don't have a Maybe namespace (class) unless the user does import maybe + maybe.from. So maybe we'll want to just do a function most likely such as to_maybe? We could call it from, but the issue then become of 'from' being a very generic name and unless the user does a fully qualified call, maybe.from, it becomes somewhat confusing if from is imported, eg from maybe import from + from(...). I hope that makes sense.
Second question is, that library, dry-python has two versions, from_value and from_optional, I'm not sure if we need both, I don't quite understand the rationale for having two. From what I'm thinking, a single function that handles the from_optional semantics (None | T -> Maybe[T]) should be sufficient

I completely agree with you. We can create a function called maybe or, as you mentioned, to_maybe. I've already created a maybe function, but renaming it is a minor issue.
I also find the dry-python decision to have two functions a bit confusing. Perhaps they intended to encapsulate None inside Some, but I don't see many real-world cases for that. In this context, from_optional seems more applicable, and we can consider going this route.


using account key for vote cache

🗃 Github Issue Or Explanation for this PR. (What is it supposed to do and Why is needed)
https://github.com/JoinSEEDS/seeds_light_wallet/issues/1049
vote cache supports multiple accounts
✅ Checklist

[x] Github issue details are up to date for people to QA.
[x] I have tested all my changes.

🕵♂ Notes for Code Reviewer
Vote cache needs to support multiple accounts
I don't think this causes Marks error but if someone used a different account it could happen
So this is more a correctness PR - the vote cache needs to store what account the vote was for.
🙈 Screenshots
👯♀ Paired with

the vote cache needs to store what account the vote was for.
It would not be more correct to erase the cache on logout and when they are aware of it (not yet implemented).

Because each account will have its cache stored and this would make the app consume more space.

the vote cache needs to store what account the vote was for.

It would not be more correct to erase the cache on logout and when they change account (not yet implemented).
Because each account will have its cache stored and this would make the app consume more space.

In the future we will allow multiple accounts.
I think initially that's more for business accounts which won't vote anyway.
I thought it's more fail-safe to store the account along with the prop id.

@gguijarro-c-chwy addressed the issues


Can't install on Ubuntu 16.04 (either :)), halts at 'node-gyp rebuild'

First of all I want to mention that websync seems to be EXACTLY what I've been searching for quite a while, so it's devastating to find out that it doesn't install in newer ubuntus :(
I'm getting similar errors as what's been reported already :

npm install
363 error Linux 4.4.0-22-generic
364 error argv "/usr/bin/nodejs" "/usr/bin/npm" "install"
365 error node v4.2.6
366 error npm  v3.5.2
367 error code ELIFECYCLE
368 error<EMAIL_ADDRESS>install: node-gyp rebuild
368 error Exit status 1
369 error Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
369 error Make sure you have the latest version of node.js and npm installed.
369 error If you do, this is most likely a problem with the hiredis package,
369 error not with npm itself.
369 error Tell the author that this fails on your system:
369 error     node-gyp rebuild
369 error You can get information on how to open an issue for this project with:
369 error     npm bugs hiredis
369 error Or if that isn't available, you can get their info via:
369 error     npm owner ls hiredis
369 error There is likely additional logging output above.
370 verbose exit [ 1, true ]

I 've tried to reinstall nodejs and npm a couple of times without making any progress, I've also tried other suggestions found on google but nothing helped.  Can't think of anything else to try... Truly grateful if anyone has any ideas :) Thank you.

Try some of the forks or docker containers :)
ons. 1. jun. 2016 kl. 22.59 skrev loppis<EMAIL_ADDRESS>
First of all I want to mention that websync seems to be EXACTLY what I've
been searching for quite a while, so it's devastating to find out that it
doesn't install in newer ubuntus :(
I'm getting similar errors as what's been reported already :
363 error Linux 4.4.0-22-generic
364 error argv "/usr/bin/nodejs" "/usr/bin/npm" "install"
365 error node v4.2.6
366 error npm v3.5.2
367 error code ELIFECYCLE
368 error<EMAIL_ADDRESS>install: node-gyp rebuild
368 error Exit status 1
369 error Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
369 error Make sure you have the latest version of node.js and npm
installed.
369 error If you do, this is most likely a problem with the hiredis
package,
369 error not with npm itself.
369 error Tell the author that this fails on your system:
369 error node-gyp rebuild
369 error You can get information on how to open an issue for this project
with:
369 error npm bugs hiredis
369 error Or if that isn't available, you can get their info via:
369 error npm owner ls hiredis
369 error There is likely additional logging output above.
370 verbose exit [ 1, true ]
I 've tried to reinstall nodejs and npm a couple of times without making
any progress, I've also tried other suggestions found on google but nothing
helped. Can't think of anything else to try... Truly grateful if anyone has
any ideas :) Thank you.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/furier/websync/issues/41, or mute the thread
https://github.com/notifications/unsubscribe/AAva1xihp3aDf3UJSAEhZue_E9cdlGf6ks5qHfKfgaJpZM4Ir-Jn
.

Your machine is trying to interpret node which is actually by defualt node-js so you need to update the binary
$  sudo ln -s /usr/bin/nodejs /usr/bin/node

got same issue, unable to install on Ubuntu 16.04
any idea or update to fix would be highly appreciated!

Same here. Looked at the forks and docker containers and nothing works.

I have tried all the other forks and docker scripts, but this error keeps coming back:
it seems the real problem is Not compatible with your operating system or architecture<EMAIL_ADDRESS>and<EMAIL_ADDRESS>
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm WARN optional Skipping failed optional dependency /watchify/chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm ERR! Linux 3.14.32-xxxx-grs-ipv6-64
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the hiredis package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs hiredis
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls hiredis
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/forge/websync/npm-debug.log