CSV Agent cannot use removed component [python_repl_ast]

Observation: df.head() is not a valid tool, try one of [python_repl_ast].
Thought: Question: What can you tell me about the csv file?
Thought: Let's start by taking a look at the first few rows of the dataframe to get an idea of what it contains.
Action: df.head()
Action Input: None
Observation: df.head() is not a valid tool, try one of [python_repl_ast].
Thought:

Hi @dosu-bot : here is the detailed output :
'\nYou are working with a pandas dataframe in Python. The name of the dataframe is df.\nYou should use the tools below to answer the question posed of you:\n\npython_repl_ast: A Python shell. Use this to execute python commands. Input should be a valid python command. When using this tool, sometimes output is abbreviated - make sure it does not look abbreviated before using it in your answer.\n\nUse the following format:\n\nQuestion: the input question you must answer\nThought: you should always think about what to do\nAction: the action to take, should be one of [python_repl_ast]\nAction Input: the input to the action\nObservation: the result of the action\n... (this Thought/Action/Action Input/Observation can repeat N times)\nThought: I now know the final answer\nFinal Answer: the final answer to the original input question\n\n\nThis is the result of print(df.head()):\n  ManufacturerItem Code Item Code Order Code                                        Description    Price\n0                 RICOH    418559     418559  IM C300 30ppm A4 Colour and Mono  Desktop Digi...  4415.96\n1                 RICOH    418572     418572  IM C300F 30ppm A4 Colour and Mono Desktop Digi...  4698.22\n2                 RICOH    418566     418566  IM C400F 43ppm A4 mono & 40 ppm Colour Digital...  5394.49\n3                 RICOH    418575     418575  IM C400SRF\xa0 43ppm A4 mono & 40 ppm Colour Digi...  6523.57\n4                 RICOH     11756      11756                                UK Power Lead 13amp     3.68\n\nBegin!\nQuestion: can you tell me how many off these items are printers ? \n  Sure! Here's the question, thought, action, observation, and final answer:\n\nQuestion: Can you tell me how many of these items are printers?\n\nThought: Hmm, let me think... I need to extract the item names and check if they contain "printer" in them.\n\nAction: Let me use the .str.contains() method to check if the item names contain "printer".\n\nAction Input: df['Description'].str.contains('printer')\n\nObservation: Let me use the .str.contains() method to check if the item names contain "printer". is not a valid tool, try one of [python_repl_ast].\nThought:  Sure! Here's the question, thought, action, observation, and final answer:\n\nQuestion: Can you tell me how many of these items are printers?\n\nThought: Hmm, let me think... I need to extract the item names and check if they contain "printer" in them.\n\nAction: Let me use the .str.contains() method to check if the item names contain "printer".\n\nAction Input: df['Description'].str.contains('printer')\n\nObservation: Let me use the .str.contains() method to check if the item names contain "printer". is not a valid tool, try one of [python_repl_ast].\nThought:  Sure! Here's the question, thought, action, observation, and final answer:\n\nQuestion: Can you tell me how many of these items are printers?\n\nThought: Hmm, let me think... I need to extract the item names and check if they contain "printer" in them.\n\nAction: Let me use the .str.extract() method to extract the item names and then check if they contain "printer".\n\nAction Input: df['Description'].str.extract('printer')\n\nObservation: Let me use the .str.extract() method to extract the item names and then check if they contain "printer". is not a valid tool, try one of [python_repl_ast].\nThought:  Sure! Here's the question, thought, action, observation, and final answer:\n\nQuestion: Can you tell me how many of these items are printers?\n\nThought: Hmm, let me think... I need to extract the item names and check if they contain "printer" in them.\n\nAction: Let me use the .str.extract() method to extract the item names and then check if they contain "printer".\n\nAction Input: df['Description'].str.extract('printer')\n\nObservation: Let me use the .str.extract() method to extract the item names and then check if they contain "printer". is not a valid tool, try one of [python_repl_ast]

@dosu-bot : Unfortunately the python_repl_ast tool is no longer available : it was in version 0.5a, but seems to have been removed

@dosu-bot : this isn't possible as the code is dynamically created by the Agent components, and must be run in the Langflow/Langchain execution context.


Remove document manager

Remove the Document manager in favor of a File Handler registry.
The registry is used to open files using AbstractFileHandler, and to create new files using file creation functions.
A convenience FileCreator class is provided and used in the example to create new files and directories.
Handling of "active widget" is deferred to a higher level, pending changes to the dock panel and the application shell.

cc @jasongrout
cf https://github.com/jupyter/jupyter-js-plugins/issues/141

ping @jasongrout, did you want to look at this before we move forward?  I will carry the changes through js-notebook and js-plugins.

Comments addressed.

Okay, this looks good for now. Thanks!


Add installer CI

Add a workflow to test the installer on various distros.

Open questions:

how to test the changes to the debian package
should the installer ci also test the nix flake?


Convert missing $and operator alias to [Sequelize.Op.and]

We merged yesterday https://github.com/mickhansen/dataloader-sequelize/pull/52
I only noticed this morning that there were actually more occurrences of it.
Should we perhaps extract [Sequelize.Op ? Sequelize.Op.and : '$and'] at the beginning of the file to avoid repeating ourselves?
Like $and = [Sequelize.Op ? Sequelize.Op.and : '$and']; And then only use [$and];

Extracting would be a nice way to solve it, but not overly important.

v1.7.2


JAVA-863: [3.0] Idempotence propagation in PreparedStatements.

Looks good to me other than the 2 small comments I had.  :+1:

:+1:
Please add:

an entry to the changelog
an item to the upgrade guide explaining that IdempotenceAwarePreparedStatement has been removed

:+1:

Squashed and rebased on current 3.0.


Program page: Filter and highlight indicators with no reported results

NOTE: This ticket is dependent on queries described in ticket #463.  An instance of Percentages Component #550
Story
When I learn the percentage of program indicators that have reported results, I want to zoom in on the indicators that do NOT have reported results, so that I can efficiently remedy the situation.

Acceptance criteria
Near the metric, there is a link that says "XX% of indicators have no results". When I click the link, the following things occur:

[ ] The list of indicators refreshes to display only the ones that meet the criteria.
[ ] The filtered list is visually connected to the metric, so that it's obvious you're looking at a subset of the total, and how that subset was determined. (Doesn't need to be a yellow box of course -- just my 2-second solution.)
[ ] The "# indicators" header refreshes to read "# indicators have no results".  I suggest we use the same warning icon up top and next to this string, to further reinforce the visual connection.
[ ] The "Add indicator" button is replaced with a "Show all indicators" button that restores the full list.

Housekeeping

[ ] Unit test written
[ ] i18n strings included

@jennyhavah  in a situation where targets aren't defined but data is (i.e., I added some data, but then deleted my periodic targets so the data is just assigned to the indicator but not bucketed by target) should it show as having reported results?

This is for the "Indicators have reported results" chart? My inclination is
to include an indicator as having reported results even if its targets were
deleted.
We have a separate metric for tracking whether or not indicators have
targets set, so it feels like adding extra criteria to the "reported
results" chart kind of muddles things. Let me know if you disagree.
On Thu, Oct 25, 2018 at 3:35 PM Cameron McFee<EMAIL_ADDRESS>wrote:

@jennyhavah https://github.com/jennyhavah in a situation where targets
aren't defined but data is (i.e., I added some data, but then deleted my
periodic targets so the data is just assigned to the indicator but not
bucketed by target) should it show as having reported results?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mercycorps/TolaActivity/issues/562#issuecomment-433228188,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgHHC_oLyJFWJ0dwGRrGnUo_aFMnmwLHks5uojyTgaJpZM4V-85b
.

--
JENNY MARX
User Experience Lead - TolaData
MERCY CORPS
tel<PHONE_NUMBER> | cell<PHONE_NUMBER> | skype jennyhavah
45 SW Ankeny St | Portland OR 97204 | USA
https://www.mercycorps.org/email-signature-creator#
Powered by the belief that a better world is possible.
Join us at mercycorps.org https://www.mercycorps.org/
Facebook https://facebook.com/mercycorps | Twitter
https://twitter.com/mercycorps

Nope, that makes sense to me, thanks!
Cameron McFee

On Thu, Oct 25, 2018 at 4:32 PM jennyhavah<EMAIL_ADDRESS>wrote:

> This is for the "Indicators have reported results" chart? My inclination is
> to include an indicator as having reported results even if its targets were
> deleted.
>
> We have a separate metric for tracking whether or not indicators have
> targets set, so it feels like adding extra criteria to the "reported
> results" chart kind of muddles things. Let me know if you disagree.
>
> On Thu, Oct 25, 2018 at 3:35 PM Cameron McFee<EMAIL_ADDRESS>> wrote:
>
> > @jennyhavah <https://github.com/jennyhavah> in a situation where targets
> > aren't defined but data is (i.e., I added some data, but then deleted my
> > periodic targets so the data is just assigned to the indicator but not
> > bucketed by target) should it show as having reported results?
> >
> > —
> > You are receiving this because you were mentioned.
> > Reply to this email directly, view it on GitHub
> > <
> https://github.com/mercycorps/TolaActivity/issues/562#issuecomment-433228188
> >,
> > or mute the thread
> > <
> https://github.com/notifications/unsubscribe-auth/AgHHC_oLyJFWJ0dwGRrGnUo_aFMnmwLHks5uojyTgaJpZM4V-85b
> >
> > .
> >
>
>
> --
> JENNY MARX
> User Experience Lead - TolaData
>
> *MERCY CORPS*
> tel<PHONE_NUMBER> | cell<PHONE_NUMBER> | skype jennyhavah
> 45 SW Ankeny St | Portland OR 97204 | USA
> <https://www.mercycorps.org/email-signature-creator#>
>
> Powered by the belief that a better world is possible.
> Join us at mercycorps.org <https://www.mercycorps.org/>
>
> Facebook <https://facebook.com/mercycorps> | Twitter
> <https://twitter.com/mercycorps>
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/mercycorps/TolaActivity/issues/562#issuecomment-433239204>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AHks6y6JHzt-Kfm9mAMhPYN916OTirovks5uokoWgaJpZM4V-85b>
> .
>

This is literally lightening fast @camgnostic! Like super sonic speed. I think it updated before I could blink.


Can't build with Mono Tarball

A build on windows or mac with either Xamarin or Visual Studio works fine.  Additionally, on Linux using any pre-built CLR package (like our Travis CI builds) also works.  However, if installing mono from a tarball using
./configure
make

Then the build fails with an error message indicating that the PCL targets cannot be found.

Okay, per this stack overflow post (http://stackoverflow.com/questions/27805245/how-to-solve-xbuild-netportable-version-v4-0-profile-profile344-issue-on-linux) it looks like the work around is to copy the files by hand from a computer that has them from a xamarin or visual studio install to one that doesn't.

Fixed by moving away from the PCLs


[gettext-libintl] Changes to allow compiling osx with osxcross

[ ] Changes comply with the maintainer guide.
[ ] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[ ] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[ ] Only one version is added to each modified port's versions file.

@MonicaLiu0311 IMO you shouldn't update the branch as long as the PR is understood as taking the wrong road. Rebuilding gettext-libintl is expensive. And this rebuild hit CI in a high-load time.

@MonicaLiu0311 IMO you shouldn't update the branch as long as the PR is understood as taking the wrong road. Rebuilding gettext-libintl is expensive. And this rebuild hit CI in a high-load time.

The remaining triplets have been cancelled, and this PR will be closed if there is no response next week.


Eureka Dashboard shows basic auth data for replicas when using security

When we register with Eureka using basic auth, those auth credentials are displayed on the dashboard for Eureka server replicas (registered,unavailable and available replicas and link for DS Replicas).
I have a PR I should be able to submit shortly to strip this data in EurekaController

Fixed in #974


Preference settings lost after every new revision of Beautify...

It would be nice if our preference settings would persist after a package update. Currently, we have to re-apply all of our custom settings anytime you push a revision. Thank you! :)

Really? This is definitely a bug, since you should not have to reapply all of your custom settings each time. However, Atom Beautify defines only the schema for the settings, it would not change them.
So this sounds like it would be a bug with either apm, which handles the updating of packages, or Settings-View, which handles the displaying and changing of settings.


AngularPrueba

Describe the bug
....
Link to the blitz that caused the error
https://stackblitz.com/run?file=src/app/app.component.ts
Steps to reproduce
....
Expected behavior
....
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
...
Platform
Browser name  = Chrome
Full version  = 100.0.4896.127
Major version = 100
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/100.0.4896.127 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 63471156,
  "usedJSHeapSize": 60535264,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 49c6d2ee58ce7c1528881cc6d46ebf056b3b42bf

Additional context
...

Closing this issue due to missing information. The link provided points to a non-existing project.


Increment versions for baremetalinfrastructure releases

Increment package versions for baremetalinfrastructure releases

API change check
API changes are not detected in this pull request.


Extra Steps For Gmail

For gmail you should put these extra steps:
Disable 2 factor authentication
Allow insecure apps.

Hello.
Are you sure 2-factor authentication needs to be disabled? It's enabled on my account, and I'm able to run backup.

Closing due to inactivity.


⚠ GitLab Server has degraded performance

In 0b1d4d5, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 7178 ms

Resolved: GitLab Server performance has improved in ef04993.


⚠ Samsung has degraded performance

In f4c1acd, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 6567 ms

Resolved: Samsung performance has improved in fbdc3a0.


[FLINK-32827][table-runtime] Fix the operator fusion codegen may not take effect when enabling runtime filter

What is the purpose of the change
Fix the operator fusion codegen may not take effect when enable runtime filter
Brief change log

Fix the operator fusion codegen may not take effect when enabling runtime filter

Verifying this change
This change is already covered by existing tests in RuntimeFilterITCase.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not documented)

cc @wanglijie95


Surprising inference of Never in CFE

For this code added in https://dart-review.googlesource.com/c/sdk/+/195260 we get surprising inference of Never:
class Bound {}

class BoundedGenericClass<A extends Bound?, B extends Bound> {}

class GenericClass<X extends Bound?, Y extends Bound> {
  method() {
    BoundedGenericClass<X, Y> class1a = new BoundedGenericClass/*<Never,Y>*/(); // <--
  }
}

I would have expected the inferred type arguments to be BoundedGenericClass<X, Y> and not BoundedGenericClass<Never, Y>.

That's an interesting one. Here how I think it works:

A and B are constrained by the downwards inference by X and Y from above: {A <: X, B <: Y}.
The extends-constraint is added to each by computing DOWN between the existing upper constraint and the bound of the type variable: {A <: DOWN(X, Bound?), B <: DOWN(Y, Bound)}.
DOWN(Y, Bound) is Y, so Y is inferred for B.
If I understand the spec correctly, the process goes as follows: DOWN(X, Bound?) => DOWN(X, Bound) => Never.

@leafpetersen Does my understanding of the situation make sense? Do I miss anything?

Interesting.  Agreed this is working to sped.  I think this is arguably a bug in the spec of lower bounds.  Basically, the rule for DOWN(T1, T2?) is assuming that the only way for a type to be a subtype of T2? is either be a subtype of Null (already eliminated) or a subtype of T2, and hence it is choosing Down(T1, T2) as the answer.  But this assumption is not correct, as your example shows (which is why in the subtyping rules we're not able to unconditionally apply the left right union rule).
I suspect there is a symmetric "bug" in the UP rules as well.
DOWN and UP are in general fairly poorly behaved, but we could try to improve them in a future release.  I think in this case that would mean defining a specific rule for type variables (as is done for UP) and moving it further up in the ordered algorithm.
A fix just for the inference case could be done by only considering the bound in the case that the otherwise inferred choice doesn't satisfy the bound.

Seems like we could have DOWN(X extends T1, T2) = X & T2 when T2 <: T1, but this might introduce intersection types in locations where we don't want them (so we'd need to specify more places where they are erased, in this case to Never, and then it doesn't help).

Note that this case come up when migrating pkg/kernel/lib/transformations/scanner.dart (by hand) in https://dart-review.googlesource.com/c/sdk/+/195240
The initial upload https://dart-review.googlesource.com/c/sdk/+/195240/1  had just added nullabilities to the type variables which led to the change in inference, only visible at runtime, so this behavior could be seen as a regression of inference.


Fix eachline link

Fix the link to the Eachline section from the Usage section

Coverage increased (+0.03%) to 97.571% when pulling 18c96762ef5eaa50049f9bd42ea3debf0f0f6da4 on mfurtak:patch-1 into d27d0e908bc92334d9541a50945ab0d06d1e9bf0 on piotrmurach:master.

Thanks 😍


Added example Input/Output to challenge

Input/Output example will make it clear what the challenge is asking the camper to do.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

This one is getting stale.

All right, I'll make the changes and tag for a review.

@RandellDawson This should be good to go.

@Manish-Giri I am OK with the changes.  I was wondering if maybe the following sentence:

The lowest number will not always come first.

should come right after the following sentence to save space:

Return the sum of those two numbers plus the sum of all the numbers between them.

Let me know what you think.  If you do not think this is a good idea, then I will go with your judgement.

Merge it.


Enchantment container (and applying enchantments to tool parts)

An enchantment container to give part types an enchantment slot would be a nice way to isolate it from the rest of the code.
This enchantment container can also specify limits, both in terms of enchantment level and enchantment combinations.

Very related to #631


CRAWLEE_LOG_LEVEL env var not supported

Describe the bug
APIFY_ vs CRAWLEE_ env vars:
if I set process.env.CRAWLEE_LOG_LEVEL locally to DEBUG - it would not show debug messages. With log.setLevel(log.LEVELS.DEBUG) it works as expected.
From @B4nan:

I guess that's because this is implemented in the logger directly. I guess we should set the log level when bootstrapping the Configuration class, definitely worth supporting CRAWLEE_LOG_LEVEL.

To Reproduce
process.env.CRAWLEE_LOG_LEVEL = 'DEBUG';
log.debug('Test')

Expected behavior
Should log Test debug message to console.
System information:

OS: MacOS
Node.js version: 18
Crwalee version: 3.0.0-beta.67

Also probably should be added to env vars guide.

Please update the guides


Change hadoop workflow key to bypass mapreduce workflow key

Frameworks such as Pig and Hive override the MRJobConfig.WORKFLOW_ID property, so we lose the azkaban project/flow information in the hadoop configuration.
This was tested locally. The azkaban metadata is correctly retrieved.

Thanks for the review @kunkun-tang ! Also tested hadoopJava type, it works fine. Also this feature is not supported in spark job type, so no need to test that.

@HappyRay this is related to this PR: https://github.com/azkaban/azkaban-plugins/pull/248
In that PR azkaban will inject the project/flow name information to the "mapreduce.workflow.id" configuration, so we can retrieve it on the YARN side.
We noticed a problem though where the Pig and Hive frameworks would override this property with its own Pig/Hive workflow ID name. So this PR is to change the configuration key which we inject the azkaban project/flow name to prevent the project/flow name from being overwritten.

Thanks @hungj. Could you add the details to the commit message when you merge?

Sure @HappyRay , will do.


Pixels for downloads

Fixes #5196
Who is this for?
Researching using the digitised collections
What is it doing for them?
Showing the size of downloadable images

Just a few stylistic tweaks from me:

remove extra spacing between pixel dimensions
remove 'full size'
Put PDF third in list
change label on PDF so it says Whole item (allow file format to be styled in same way as JPG

This image (0000x0000 pixels) JPG
This image (760x1000 pixels) JPG
Whole item PDF

Some suggestions from me from having a quick look around. I think the image sizes are easier to read with the space but I've uploaded a version without too.

On Friday I asked Holly to have a look at this to sense-check, as someone who handles image-size related calls to the Digital Production team. Can we hold off til she gets back to me, please?


how to create return PaymentRequest::make(Order::class) Order class?

im following the doc for #Use Payment Action
but im getting
Class "App\Filament\Cliente\Resources\Order" not found

Order::class can be any class connected to the payment, like if you want to charge customer wallet on this case you need to use Wallet::class where is the Model of the wallet

What if i dont have any class?

i think you must have a class, to handle your callback we handle it all to attach a class


Add hooksList Method

You who worked hard, add this too.

\Bigbluebutton::join([
'meetingID' => $meeting_id,
'userId'=>$user->id,
'userName' => $user->name_ar,
'password' => $password,
'joinViaHtml5'=> true,
'avatarURL'=>$user->AvatarURL,
'webVoiceConf'=> [
'bbb_auto_join_audio'   => true,
'bbb_listen_only_mode'   => false,
],
'customParameters' => ['userId' => $user->id]
]);
cannot view user image and not joining the audio bridge automatically
what is problem ?


test for array#shuffle fisher-yates implementation

See the inline code comments re: "how does this test prove the Fisher-Yates algorithm is used"?
I tried to avoid adding sinon (to dev dependencies), but I don't know of a better way to selectively mock Math.random().

👍


document.registerElement deprecation

Hi,
With Chrome 70 release;
"[Deprecation] document.registerElement is deprecated and will be removed in M73, around March 2019. Please use window.customElements.define instead. See https://www.chromestatus.com/features/4642138092470272 for more details." warning is started to apper on console.
Regards

This is already fixed in the core, but currently document.registerElement is still present for older browsers. I will remove it entirely in the next version.


[#3122] Replace lodash.trimend import by lodash v4.17.21

Fixes #3122
Description
This PR removes the individual imports of lodash modules to use the latest version of the lodash package avoiding security vulnerabilities.
Specific Changes

Removed every individual module of lodash in the package.json files.
Modified every import of the individual modules of lodash.

Testing
The following images show the build and the tests running successfully.

Replaced by #3125


computed_field with MissingType in return type: PydanticSerializationError: Unable to serialize unknown type:

Hi - this is a very nice lib! Just reporting something I saw
from pydantic_partials import PartialModel, MissingType, Missing

from pydantic import computed_field

class Test(PartialModel):
    a: int

    @computed_field
    @property
    def b(self) -> int | MissingType:
          if a < 2:
              return Missing
          return a

t = Test(a=1)
t.model_dump_json()

raises
.local/lib/python3.11/site-packages/pydantic/main.py", line 415, in model_dump_json
    return self.__pydantic_serializer__.to_json(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
pydantic_core._pydantic_core.PydanticSerializationError: Unable to serialize unknown type: <class 'pydantic_partials.sentinels.MissingType'>

Instead, I would expect it to drop the computed_fields.
This is nit.
Versions
pydantic-partials: 1.1.0
pydantic: 2.8.2

@thibaut-lo Thank you for the report, very much appreciated!
I'll try and take a look at this be end of the week, see if I can get it fixed.  I don't think I've tested the library with computed fields, so it's probably something I am not taking into account that I should have.  I'll be sure to add a unit-test when I fix it.
Thank you!

Thinking about it for a few minutes, it's probably related to the biggest limitation I have right now with Pedantic. I can't raise a PydanticOmit in a serializer without Pedantic producing an error (See https://github.com/pydantic/pydantic/discussions/5461#discussioncomment-7411977).
I actually delete/prevent Missing values from being assigned to attributes right now as a work-around.  A computed field however can return a value and I can't prevent it by deleting the attribute (since it's computed field property function).
When I get some time later this week to look into this, I'll have to see if I can figure out a workaround for this use-case, until there is some way to tell Pedantic not to serialize something from the serializer method.

@thibaut-lo Sorry, I have not had a chance to see if there is a workaround (had a family emergency over last weekend).
There might not be any good workaround, until PydanticOmit can be used in a serializer (see above).  The Pydantic people seem interested in eventually doing that, but it's not a high priority for them at the moment.
Perhaps I can override the model_dump method and try to deal with it there.  It won't fix model_dump_json, and I would have to recursively look the dump to find the Missing values and remove them.
So it's not ideal and a bit messy, but it may support your use-case well enough for now to be worth doing.  Like I said, I'll see if I can do this later this week.  We will see.

@thibaut-lo The good news is it sounds like there is a new feature I the next release of Pydantic (v2.11) that will allow one to conditionally exclude fields via a user-provided function.  I could probably use this to do what is needed.  As soon as the v2.11 is out, I'll attempt to utilize that new feature and get your use-case working.
Today, I was actually going to try modifying the base-model and override model_dump to at least get that specific function working with your use case.  But in light of this soon to be released feature in v2.11 of Pydantic, I'm going to wait for that and see if that new feature is good enough for your use case here.
Comments About Code
If it turns out that v2.11 can't be used for your use case, I'll serasouly consider adding your code (with some tweaks) to the PartialModel.
Just a few side comments about the code you provided (in case it's helpful):
The hard thing about model_serializer is that the super-class's version won't be called if a subclass does its own model_serializer.  This is one of the reasons I decided to not go this route in the first place.  Also, there could be private fields (so the self.__dict__.keys() might include other attributes that don't matter).  You would be better off using the ModelCls.model_fields property to get the Pydantic-aware fields and use that key-list instead.

Forgot to say thanks for both feedbacks! The new exclude-if pydantic feature sounds promising! Thanks again.


Error when running scripts/run_local.sh

Output from my machine:
~/libsmelt# bash scripts/run_local.sh
SMLT_MACHINE=XXX
SMLT_HOSTNAME=XXX
MODEL_ROOT=model
Creating model for XXX. logfile: model/create_model.log
checking simulator in model/Simulator
checking likwid in model/likwid
building bench/pairwise...OK.
gathering data...
   - running lscpu on sc2-hs2-b1641
   - running cpuid on sc2-hs2-b1641
   - running proc/cpuinfo
   - running likwid on sc2-hs2-b1641
/usr/bin/lua5.2: error loading module 'liblikwid' from file '/usr/lib/x86_64-linux-gnu/liblikwid.so':
	/usr/lib/x86_64-linux-gnu/liblikwid.so: undefined symbol: likwid_hwloc_set_area_membind_nodeset
stack traceback:
	[C]: in ?
	[C]: in function 'require'
	/usr/share/lua/likwid.lua:33: in main chunk
	[C]: in function 'require'
	/usr/bin/likwid-topology:34: in main chunk
	[C]: in ?

If you have a local installation of likwid it conflicts with the library that the scripts installs.
https://github.com/libsmelt/libsmelt/blob/98746b7d52fe475ab86934262813296cafbd0eb2/scripts/run_local.sh#L77
Should be ./model/likwid/likwid-topology


EvmError: InvalidOperandOOG when called SystemContractsCaller.systemCallWithReturndata

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.0.2 (fb6d0c5 2024-08-08T00:23:13.752629000Z)
What command(s) is the bug in?
forge test --match-test "testComputeCreate2Address" --system-mode=true --zksync --gas-limit<PHONE_NUMBER>--chain 300 -vvv --via-ir
Operating System
macOS (Apple Silicon)
Describe the bug
I created the test case regarding create2 following zkSync way.
L2ContractHelper.computeCreate2Address works fine.
But when I called SystemContractsCaller.systemCallWithReturndata, it returned InvalidOperandOOG
I've set is_system = true in foundry.toml.
Is there any hint to resolve it?
The full log is here. Thank you.
https://gist.github.com/wshino/c13c05afeff62c52fae0d029bd6077bf

Hey, thanks for the report! Could you share here the testComputeCreate2Address to better understand the console logs? Thank you!

@Karrq Ah sorry it's my bad. The following link is the implementation.
https://github.com/zkemail/ether-email-auth/blob/30058943b7b286dc726614d43eb10173da80df6e/packages/contracts/test/ComputeCreate2Address.t.sol
First I get the computed address via L2ContractHelper.computeCreate2Address.
Until now, deployments were performed in the same way as for normal EVMs. In this way, it was possible to verify that the calculated address and the deployed contract's address matched. However, this did not seem to work correctly in production, so I changed to an implementation using SystemContractsCaller.
But when I try to deploy the contract via SystemContractsCaller. The InvalidOperandOOG error happened the following line.
        (bool success, bytes memory returnData) = SystemContractsCaller
            .systemCallWithReturndata(
                uint32(gasleft()),
                address(DEPLOYER_SYSTEM_CONTRACT),
                uint128(0),
                abi.encodeCall(
                    DEPLOYER_SYSTEM_CONTRACT.create2,
                    (
                        accountSalt,
                        bytecodeHash,
                        abi.encode(
                            address(emailAuth),
                            abi.encodeCall(
                                EmailAuth.initialize,
                                (
                                    recoveredAccount,
                                    accountSalt,
                                    address(this)
                                )
                            )
                        )
                    )
                )
            );

I have not tried it yet, but I have heard that it works fine with zksync-hardhat.

Is there any updates?

Hi! Sorry for the long wait.
After debugging, I found that the issue is due to SystemContractsCaller being a library, therefore the call environment in which the system call happens is NOT a zkSync environment, which leads to this error.
The way I fixed the issue locally was thru a factory contract, which would then be 100% run in zkVM (here's a patch for your test to use this pattern).

Looking at the test I can see that previously you had used the usual approach for the deployment (via new) and for all intents and purposes that should have been sufficient (and it would also work nicely in a script, where you would be able to broadcast this deployment): could you elaborate on the failure you encountered (the comment "not working actually by clave side")?
I also tried to understand the context of your test, and found that you were having issues with the bytecode size limit, and had to switch to slower compilation because of that - I suggest looking into creating a separate foundry profile, which would allow you to specify entirely different configuration for zksync, like different folders for these sources, so that you would only need to use via-ir with this second profile

Thank you for your reply and details. Indeed the previous logic it works fine. But the problem is that the calculated address value is different from the actual deployed address. That's the reason I updated the logic. Fortunately the test case using SystemContractsCaller it works fine with hardhat zksync.
Currently we froze our contract code for audit. Next time to fix some vulnerability and audit report issues, I will modify this logic through factory contract. Thank you so much.

Okay I see, I'll check and get back to you why you might be observing this, considering [we have a test] for the create2 address computation (https://github.com/matter-labs/foundry-zksync/blob/543f39e48a4cffc696c7c955a68448c6675689ca/testdata/zk/Contracts.t.sol#L226).
Yes, with hardhat it would work because the hardhat deployment generates transactions directly, meanwhile foundry runs the script written in solidity and collects any external call as a broadcastable tx, which is then broadcasted

Hey @wshino I added a new test covering your usecase, but I wasn't able to reproduce your error with it, so I'm not sure why you are getting a different address than expected.
Please let me know if you still encounter this issue, but otherwise I'll go ahead and close it for now since the original has been resolved (or at least a workaround is known).
Thanks!

@Karrq Thank you for your test case and trying to reproduce it. Actually the different deployed address problem couldn't be reproduce with foundry zksync, it only happened on zkSync sepolia testnet. Perhaps it can be reproduced on zkSync maininet too.
Anyway I'm going to implement the above logic through factory contract. it can be solved InvalidOperandOOG error.


Carousel spec

Uploaded carousel spec. This closes issue #6438
Microsoft Reviewers: Open in CodeFlow

this should be consolidated with this -> https://github.com/microsoft/AdaptiveCards/blob/feature/carousel/specs/DesignDiscussions/Carousel.md


Enable openzwave's network heal and soft reset.

Description:
Makes Zwave's network heal and the controller's soft reset
commands available as service calls. They can be used
in automation to help keep the zwave network healthy.
An example use in the automation section of configuration.yaml:
    automation:
     alias: At 2:35am, heal problematic zwave network routes
     trigger:
       platform: time
       after: '2:35:00'
     action:
       service: zwave.heal

Checklist:

[x] Local tests with tox run successfully.
[ ] TravisCI does not fail. Your PR cannot be merged unless CI is green!
[x] Fork is up to date and was rebased on the dev branch before creating the PR.
[x] Commits have been squashed.
If code communicates with devices:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Looks good :dolphin: Thanks


Tm 58 fe buttons and api

Added two buttons for accepting or rejecting an application
The button are visible only for the application in 'pending' status
When a decision is taken the teacher is navigated back to the list of application with a message confirming the successfull operation

its is just the professor that can accept/reject, i'll fix that


Windows arm64 bots in devicelab are restarting during tests

For example: https://ci.chromium.org/ui/p/dart/builders/ci.sandbox/vm-win-debug-arm64/1115/infra
per @aam the system logs say:
The process C:\WINDOWS\SYSTEM32\shutdown.exe (FLUTTER-WIN-19) has initiated the restart of computer FLUTTER-WIN-19 on behalf of user flutter-win-19\swarming for the following reason: No title for this reason could be found
 Reason Code: 0x800000ff
 Shutdown Type: restart
 Comment:

And the reboots seem to coincide with
Successfully scheduled Software Protection service for re-start... Reason: RulesEngine

in Windows logs.

@loic-sharma are you familiar with the "Software Protection service"? Should we disable this on our arm bots, per https://answers.microsoft.com/en-us/windows/forum/all/successfully-scheduled-software-protection-service/24d9cc65-9fb1-4653-a5be-133905371aaf
Also, we recently had a power outage in the devicelab, I wonder if that could have reset some setting?
FYI @kentnguyen99 (but it's unclear what action item we have at this point).

@loic-sharma are you familiar with the "Software Protection service"? Should we disable this on our arm bots, per https://answers.microsoft.com/en-us/windows/forum/all/successfully-scheduled-software-protection-service/24d9cc65-9fb1-4653-a5be-133905371aaf

I'm not familiar with the "Software Protection service" off-hand. The web page you linked has this comment which indicates it's probably not a good idea to disable:

[Software Protection service] Enables the download, installation and enforcement of digital licenses for Windows and Windows applications. If the service is disabled, the operating system and licensed applications may run in a notification mode. It is strongly recommended that you not disable the Software Protection service.

Is this a common source of restarts? If it's very infrequent, perhaps it causes an acceptable level of flakes?

Also, we recently had a power outage in the devicelab, I wonder if that could have reset some setting?

I believe this is our guide to setup a Windows machine, AFAICT it doesn't mention anything directly related to Software Protection service settings: https://g3doc.corp.google.com/company/teams/flutter/infrastructure/devicelab/windows_host.md?cl=head


Windows Service via NServiceBus.Extensions.Hosting/BackgroundService

Is it supported to use NServiceBus.Extension.Hosting/BackgroundService for a windows service endpoint? Is there any special reason that the dotnet templates aren't using these?
Thanks

Hi @Tobias-Gr

OK. Is there a sample/documentation how to use NServiceBusExtensions.Hosting with the default .NET Core 3.1 worker-template?

Not yet, we're in the process of upgrading the whole CI infrastructure to enable that. Once the upgrade is done, samples will be made available online.

Do I need an own implementation of the BackgroundService-class or does the IHostedService-implementation in UseNServiceBus do the whole job on its own?

IIRC, I wasn't involved in the work that created that package, everything is done automatically.
@timbussmann can you confirm?

UseNServiceBus register its own IHostedService which will start and stop the endpoint as part of the generic host lifecycle, so there is no need for an additional hosted service (BackgroundService is just an implementation of IHostedService). Does that help?

Yes, thanks.
A sample (when ready) would be helpful anyway, especially regarding encapsulation of a common endpoint configuration for all endpoints. I thaught of having a Host base class (based on the one in the old NSB dotnet templates) with a BuildEndpointConfiguration method that I pass as a func to the UseNServiceBus method. But maybe you have other ideas.

@Tobias-Gr not sure whether I understand you correctly, are you thinking about hosting multiple endpoints in the same generic host?
Creating samples on using the generic host in different scenarios is on our backlog. Currently, the samples are focused on integrating with ASP.NET Core 3 which is the most common use case for the generic host.

It's not about multiple endpoints in one generic host but about sharing a common endpoint configuration in different endpoints (each in its own generic host).
It's OK for me to wait for the samples.

@Tobias-Gr I left a note for the people working on samples to update this issue as well when they are released.

@Tobias-Gr There is a PR that shows how the generic host can be leveraged that hopefully will end up being available soon as a sample for downloading
https://github.com/Particular/docs.particular.net/pull/4724

The sample is no available at
https://docs.particular.net/samples/hosting/generic-host/

Thanks!

FYI we are also working on updating the templates https://github.com/Particular/dotnetTemplates/pull/43
as a first iteration it will not contain MS logging. We are still working on a good logging integration and would ship that later as an update. But as a first iteration the templates will switch to generic host


UI uses new pending process

In UI keep track of object status remove pending.
example:

user creates order, UI calls SDK to create order (UI handles submission failures)
SDK emits log for all pending transactions with info.
based pending info, UI adds order in state with status as pending
when tx confirms (pending tx gets updated minus the confirmed tx), UI gets a log for OrderCreated and replaces the order in state.

if user is on market trading page for the market the whole order is refreshed.

UI notes on orders:
building the order book based on pening tx.
order book only gets created/updated when user is on the market trading page.
log emitted that affects the orderbook
cancel, filled, created.
signed Transactions the UI is interested in (list not complete)
Misc:
ReputationToken.faucet
Cash.faucet
Trading:
Trade.publicFillBestOrder
Trade.publicTrade
CancelOrder.cancelOrder
ClaimTradingProceeds.claimTradingProceeds
CompleteSets.publicSellCompleteSets
CreateOrder.publicCreateOrder
Market.finalize
Reporting:
DisputeWindow.buy
DisputeWindow.redeem
Market.contribute
Market.doInitialReport
Universe.redeemStake
InitialReporter.redeem
DisputeCrowdsourcer.redeem

@bthaile -- is this in progress w/ the stuff on Monday about submitting txns?


The tooltip in the drag-and-drop area for uploading an image is not properly translated

When the mouse cursor is placed over the drag & drop zone to insert an image into content, a tooltip is displayed with the text "No file chosen".
As the area disappears once a file has been chosen, it's irrelevant to specify the status. What's more, the text is in English even when the user's preferences are in French.

Steps to reproduce

Select the "Insert image" button in the Froala bar
Place mouse cursor over "Click to browse" area.

Observed result in a french page
A tooltip with the text "No file chosen"
Expected result in a french page

A tooltip translated with the text "Aucun fichier sélectionné"
or
No tooltip

I could use a way to hide the tooltip or to change the text as well, even without the need for translation.


Updated images on environmental variable page

What are you changing in this pull request and why?

🚀 Deployment available! Here are the direct links to the updated files:

https://docs-getdbt-com-git-nfiann-img-sweep-dbt-labs.vercel.app/docs/build/environment-variables

PR description:
I've updated the images on the environmental variable page as part of the UI project
I've updated the images under the following sections:

Overriding environment variables at the job level (the first image)
Overriding environment variables at the personal level (the three images under this section)


haystac database creation fails

Hi,
I installed haystac through conda like it was outlined on your GitHub, but when trying the example command (haystac database --mode build --query '"Yersinia"[Organism] AND "complete genome"[All Fields]' --output yersinia_example) of building a database the program crashes:
"[Wed Dec 21 10:53:23 2022]
Job 6: The number of index chunks needed for the filtering alignment are being calculated.
Downstream jobs will be updated after completion.
Not a conda environment: /home/haystac/cache/conda/a8748482f2d8c2ef49c84c3e2d1baa35
Not a conda environment: /home//haystac/cache/conda/a8748482f2d8c2ef49c84c3e2d1baa35
Not a conda environment: /home//haystac/cache/conda/a8748482f2d8c2ef49c84c3e2d1baa35
Not a conda environment: /home//haystac/cache/conda/a8748482f2d8c2ef49c84c3e2d1baa35
[Wed Dec 21 10:53:23 2022]
Error in rule bowtie_index_accession:
jobid: 43
output: /home/haystac/cache/ncbi/Yersinia_enterocolitica/NZ_HF571988.1.1.bt2l, /home/haystac/cache/ncbi/Yersinia_enterocolitica/NZ_HF571988.1.2.bt2l, /home/haystac/cache/ncbi/Yersinia_enterocolitica/NZ_HF571988.1.3.bt2l, /home/haystac/cache/ncbi/Yersinia_enterocolitica/NZ_HF571988.1.4.bt2l, /home/haystac/cache/ncbi/Yersinia_enterocolitica/NZ_HF571988.1.rev.1.bt2l, /home/haystac/cache/ncbi/Yersinia_enterocolitica/NZ_HF571988.1.rev.2.bt2l
log: /home/haystac/cache/ncbi/Yersinia_enterocolitica/NZ_HF571988.1_index.log (check log file(s) for error message)
conda-env: /home/haystac/cache/conda/a8748482f2d8c2ef49c84c3e2d1baa35
shell:
bowtie2-build --large-index --threads 4 /home/haystac/cache/ncbi/Yersinia_enterocolitica/NZ_HF571988.1.fasta.gz /home/haystac/cache/ncbi/Yersinia_enterocolitica/NZ_HF571988.1 &> /home/haystac/cache/ncbi/Yersinia_enterocolitica/NZ_HF571988.1_index.log
(one of the commands exited with non-zero exit code; note that snakemake uses bash strict mode!)"
Do you have any other ideas on how to fix this?
Thank you for your help!
Best,
Nicola

Hi Antony,
Happy new year to you, too! I should have closed this issue sooner. My apologies for that! I got everything to work two days before your answer. I am not 100% sure what the issue was, but when I created a new conda environment in a new directory, all the errors disappeared.
Anyways, thank you for your reply!
All the best,
Nicola


Only apply indexed access write simplifications to types that arise from mutation

This is an attempt at alleviate some additional errors introduced by #30769. The write simplifications introduced there were applied whenever a type was related to an indexed access type, irrespective of whether that type denoted a real update or just a value of lookup-type.
This PR only applies the write simplifications to indexed access types that arise through assignments to element access expressions.
Fixes #31833
Fixes #32816
A related digression: there are code paths that do not get write-simplified (with and without this PR) which introduce unsoundness. Is this by design?
const foo = { x: { a: true }, y: { b: 42 } };
type Foo = typeof foo;

const bar = { x: { c: false }, y: { d: 100 } };
type Bar = typeof bar;

function example(arg: Foo | Bar, k: "x" | "y") {
    // The union Foo | Bar is not simplified
    // to an intersection under indexed access
    // even though arg[k] conceptually has the
    // type: (Foo | Bar)["x" | "y"]
    arg[k] = { a: true, b: 100 } // no error, but what if arg is Bar?
}

This approach looks fine to me, but we'll need to sync it and run the extended test suites on it. Also ping @ahejlsberg since I imagine he'll be interested in it.

@weswigham, will do! FWIW, very few people seem to have stumbled upon this issue so maybe it isn't worth the additional complexity.

Unfortunately, we never finished reviewing this PR. It is pretty old now, so I'm going to close it to reduce the number of open PRs.


Shield Activity Support

This PR is needed to support shield activity on MPW, it is divided in 4 main parts:

Add the interface SimplifiedNote which is the structure we are actually going to send to MPW
All nullifiers belonging to the wallet are stored in a map f: nullifier --> SimplifiedNote. In this way we can decrypt a ShieldSpent only by its nullifier.
Add a function decryptTransactionOutputs which tries to decrypt the ShieldOutput of a transaction and returns their simplified version
Make handle block return the list of transactions that belongs to the wallet

rebased to get rid of the git merge commit. You can check that the diff is trivial with
git diff 438fb04
and check the files modified by this PR


Gssoc'21 Adding Form Validation And Error/Success message in Contact page

Hello @chiraag-kakar sir i am a gssoc'21 participant and i want to add some form validation in contact page because I think there are no validations in it as i am able to fill all fields by 'a' and it got submitted successfuly and some flash message feature using message framework in Django Kindly Assign me this issue I will do this as soon as possible.

Can I start working on it @codeit13 sir

Comment on #111 and close this.
Don't create same issues again. @arpit456jain
Comment your approach as well to have better chances of getting assigned.

ok mam


Fix handle leaks in encomsp and remdesk server

Ensure proper thread cleanup when these channel servers are stopped.

@freerdp-bot can you test that please ?


Freezes

TogglDesktop 7.2.219 always freezes when attempting to stop timer or delete existing time entries.

Thanks for reporting the problem!
This is a known problem, and we have been working on sorting this out the last few days.
We have a new build you can try available here:
https://assets.toggl.com/installers//TogglDesktopInstaller-7_2_261-2015-08-11-23-59-06.exe
It would be great if you could let us know if this one works better.
(closing this as duplicate of #1444)


Testing aliases

Hello,
it would be great to test if a method is an alias of another; someting like this:
describe Array do
  describe '#[]' do
    it 'is an alias of #fetch' do
      ...
    end
  end
end

But I don't know if it is possible; I don't even know if you can know if a method is an alias of another :-/

I have a lot to say about this, but the short version is I don't think we can do it, and I wouldn't want to even if we could.
In cases like this I want to specify that the behavior of two methods is the same. You picked an unusual case because fetch and [] have different syntax, but it's more common that aliases have the same syntax. In those cases, I use a shared example group: https://www.relishapp.com/rspec/rspec-core/docs/example-groups/shared-examples or a list of names: https://github.com/rspec/rspec-core/blob/46485e1fe4d40a32bf76f5ac6b3e9974ea5a85cf/spec/rspec/core/example_group_spec.rb#L630-636.
HTH,
David


[Protection Level] Page Item Tracking Lines - procedure SetControls

Hi Team,
at the moment the protection level is local, but I would like to access it from my PageExtension.
Would be great if you could change the protection level.

Already made protected


CSRF problem using Sprig on a loop of Wishlist (Verbb) add/remove from list or products 'add to cart.

Describe the bug
We are having issues may be something we are missing so apologies if it is. :-)
We get a CSRF problem using Sprig more than one form/button of Wishlist (Verbb) add/remove from list or products 'add to cart (Craft Commerce).
If you click the button to add/remove it will work for one of two buttons, in the browser console network tab it shows 400 Bad Request, and the response is  'Unable to verify your data submission.' This points to CSRF; if we turn off CSRF in the .config ('enableCsrfProtection' => false in the config) it all works fine again.
It happens only on the first load of the page eg for new users; if you reload the page, it works fine.
We have done a lot of testing... both on our server and on a new server installed with only Craft, Sprig, Blitz, Wishlist plugins. Only blitz/remove index.php in the .htaccess and basic version of the templates.
First though it was a Blitz/Sprig thing, but it happens if Blitz is off too.
If we do a Sprig call in a loop or even just two calls like below, it does not work on the first load:
main.twig
<!doctype html>

      <meta charset="UTF-8">
      <meta name="viewport" content="width=device-width, initial-scale=1.0">
      <title>Sprig / Blitz / Wishlist TEST</title>

        Sprig / Blitz / Wishlist TEST

                {# normaly a loop #}
                    Home
                    {{ sprig('_components/wishlist', {'entryId': 2}, {'s-trigger': 'load'}) }}

                    News
                    {{ sprig('_components/wishlist', {'entryId': 62}, {'s-trigger': 'load'}) }}
                {# normaly a loop end #}

    {{ sprig.script }}

_components/wishlist.twig
{% if sprig.isRequest  %}

    {% set item = craft.wishlist.item(entryId) %}
    {% if item.inList %}
      {# Posts to the `remove` action on click #}
      <form sprig s-action="wishlist/items/remove" s-method="post" name="removeForm_{{ entryId }}" id="removeForm_{{ entryId }}">
          <input type="hidden" name="elementId" value="{{ entryId }}">
          <button class="inline-flex items-center px-3 py-2 border border-transparent text-sm leading-4 font-medium rounded-md shadow-sm text-white bg-red-600 hover:bg-red-700 focus:outline-none focus:ring-2 focus:ring-offset-2 focus:ring-red-500">
            Remove from wishlist
          </button>
      </form>
    {% else %}
      {# Posts to the `add` action on click #}
      <form sprig s-action="wishlist/items/add" s-method="post" name="addForm_{{ entryId }}" id="addForm_{{ entryId }}">
          <input type="hidden" name="elementId" value="{{ entryId }}">
          <button class="inline-flex items-center px-3 py-2 border border-transparent text-sm leading-4 font-medium rounded-md shadow-sm text-white bg-green-600 hover:bg-green-700 focus:outline-none focus:ring-2 focus:ring-offset-2 focus:ring-green-500">
            Add to wishlist
          </button>
      </form>
    {% endif %}

{% endif %}

To reproduce
Steps to reproduce the behaviour:

Make sure the pages are cached by checking the Blitz comment at the bottom of the HTML source.
The way we get it to break is to open Firefox (Clear the cache in the preferences) It's the same on Chrome/Edge too.
Enter the URL, on the first load only one of the forms/buttons will work. The other will give a 400 error.

Expected behaviour
Expected both buttons to work. :-)
Versions

Plugin version: Sprig 1.11.0, Blitz 3.11.1, Wishlist 1.4.11
Craft version: 3.7.33 - also tried on 3.7.28, which had the same problem (before Craft CMS 3.7.33 - 2022-02-15, which notes: The way CSRF tokens are generated has changed in this release, so all users will be logged out during the update.)

I think this may be to do with how Craft generates CSRF tokens for users the first time they hit the site. The simplest way to resolve this should be to switch from one Sprig component per iteration of the loop, to one encompassing Sprig component that contains the entire loop. Can you test if that solves the issue?

So, what happens when you do that?

I get the 400 error on some of the buttons again there will always be one that works, which one is random may be down to what loads first/last.

Sorry about this Ben let us do some more testing here as it seems to be working on a few tests here now the encompassing Sprig component way. Just for other projects is it best to keep away from the loops with Sprig then?

Hi Ben,
Sorry to resurrect this thread but I'm having the same issue. I can email you a link where you can recreate the issue if that helps?
I don't think it's possible to easily create a single all encompassing sprig component though in my situation, so if you had any ideas it would be massively appreciated!
Everything was working fine, until I turned on Blitz static caching, then had to add 's-trigger': 'load' to the looped sprig components. for them to work. Then, as per g17, when a 'fresh user' try to add a product to the cart, I get the 400 error, unless I can find the 'last' sprig item that loaded. If I refresh the page, all components work fine, so it's just on this first load of a product page.
Thanks in advance!
Rob

You're right that this is still an issue for first requests. I've created a dedicated issue, along with a workaround, at https://github.com/putyourlightson/craft-sprig/issues/279.


NPE on non-OK responses

If a response doesn't contain a ContentType (e.g. for 204 responses), processResponseTags will fail with an NPE.

Fixed in b0dcccd4ab8216e4e79864e2654c6554b8a0c9dd


[203] Clarification on NEAR's MPC Service

Title:
Clarification on NEAR's MPC Service
Description:
Instead of "NEAR's MPC service allows for nodes to safely join and leave, without needing to re-derive a master key" would it be more clear to say "However, each node in NEAR's MPC service contains one piece of a master key for a given account, and supports nodes entering and leaving without  needing to re-derive a master key".   There is still a gap in explaining "how" Near's MPC can avoid re-deriving a master key, but is this language accurate and more helpful?
Summary:
Customer is seeking clarification on the explanation of NEAR's MPC service. Consider providing more detailed information on how it avoids re-deriving a master key.
Info
Reported by: charles<EMAIL_ADDRESS>- Open in Gleap
Location: US
Type: 🚨 BUG
🔗 Open share link
🌍 Open ticket in Gleap
Metadata
browserName: Chrome(<IP_ADDRESS>)
userAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
browser: Chrome
systemName: MacOS
sessionDuration: 7923
devicePixelRatio: 2
screenWidth: 1512
screenHeight: 982
innerWidth: 1369
innerHeight: 823
currentUrl: https://docs.near.org/concepts/abstraction/chain-signatures
language: en-US
sdkVersion: 10.1.0
sdkType: javascript
environment: prod

but they do not contain a piece of the master key, as the text explain, they have a signature share.
No master key is derived at any point, otherwise an adversarial node could leave and join iteratively until getting enough pieces of the master key to reconstruct it.
At least this was my understanding


Restart of the server required after changing the config file ?

Hi,
after changing the config file .i.e adding the Keyring for all my attached nodes do I have to restart the server and if yes - how do I need this as I only found an option to "Start" the server but not how to "Stop or Restart"
Best, Michael

Hi @gekko68
Yes you need to re-start the server in order to read the new configuration. At this point there is no clean way to stop the server. We are currently refactoring the multiprocessing code which should solve this.

Closing since this issue is covered under issue #662


Approach to encode sequence position in DOM is rather fragile.

If the sequence id is '123456', span element are given the id '123456_n' where 'n' is [0, sequence length - 1]. And position is inferred later (like in L1294) by reading after _.
This breaks for the simplest of the use cases. For example if sequence id itself contains an _, which is quite common.
Workaround is to not pass sequence id to Sequence constructor and disable title, or strip sequence id of _ before passing to Sequence.
A possible solution is to use data- attributes to remember position.

This component works with protein sequences. Rather than a protein id, it is meant to get a protein accession as defined by UniProt. In UniProt accessions do not have "". Still, an easy fix is to localize the last "" in the span element id, rather that the first one.

Can this component not be used with nucleotide sequences?

Potentially, yes, there is no restriction on whar the sequence actually
represents.  However, I do not recall whether it was tested with such long
sequences. Have you try it with nucleotide sequences? If so, please let us
know how it works for you, and how useful it is in its current state.
On Jan 21, 2015 9:55 AM, "Anurag Priyam"<EMAIL_ADDRESS>wrote:

Can this component not be used with nucleotide sequences?
—
Reply to this email directly or view it on GitHub
https://github.com/ljgarcia/biojs-vis-sequence/issues/12#issuecomment-70811020
.

Yup. We are using biojs-vis-sequence for SequenceServer.
We are working on a complete revamp and decided to use biojs-vis-sequence for visualising sequences. Here's a demo - http://antgenomes.org/sequenceserver/.
Use
>SI2.2.0_06267 locus=Si_gnF.scaffold02592[1282609..1284114].pep_2 quality=100.00
MNTLWLSLWDYPGKLPLNFMVFDTKDDLQAAYWRDPYSIPLAVIFEDPQPISQRLIYEIR
TNPSYTLPPPPTKLYSAPISCRKNKTGHWMDDILSIKTGESCPVNNYLHSGFLALQMITD
ITKIKLENSDVTIPDIKLIMFPKEPYTADWMLAFRVVIPLYMVLALSQFITYLLILIVGE
KENKIKEGMKMMGLNDSVF

against Solenopsis invicta predicted transcripts 2.2.3 database to get a guaranteed hit. Scroll down to a hit and select 'View Sequence'.
We disable visualization for sequences bigger than 10k chars. Will probably up the number after some experimenting.
The current feature set is looking very useful for our needs. #7 is a major showstopper for us though.

Hey does 2e51011a27a5c36d13882955881b4786fb387d22 fix your problem?

Let's wait for Anurag saying whether it works or not. But, are we sure it
does not break anything? I kind of like more looking for the last "_"
rather for the first one, because the last one for sure was the one added
with the sequence position.
On Thu, Jan 22, 2015 at 11:52 PM, Seb<EMAIL_ADDRESS>wrote:

Hey does 2e51011
https://github.com/ljgarcia/biojs-vis-sequence/commit/2e51011a27a5c36d13882955881b4786fb387d22
fix your problem?
—
Reply to this email directly or view it on GitHub
https://github.com/ljgarcia/biojs-vis-sequence/issues/12#issuecomment-71123764
.

It works. Thanks.
I had mentioned in my first comment that not passing id to the widget works, and I had been doing just that to work around the issue till now. I realise now that if two instance of Sequence widgets were initialised on a page, the workaround would have created more than one element with the same id on the same page. Duplicate id for span elements doesn't get in the way of working of the widget, but is semantically incorrect. @greenify's commit takes care of that.

@ljgarcia In the broader scope of things, where the widget is not just confined to Uniprot sequences, many funky characters may be present in the sequence id. I wonder if deriving id for DOM elements using sequence id (by adding _n to them) would be a good idea in that case.


--arg-file config YAML is never found

I'm attempting to put all of my environment variables in a YAML as described in the --help section of ACA-Py:
Args that start with '--' (eg. --admin) can also be set in a config file
(specified via --arg-file). The config file uses YAML syntax and must
represent a YAML 'mapping' (for details, see
http://learn.getgrav.org/advanced/yaml). If an arg is specified in more than
one place, then commandline values override environment variables which
override config file values which override defaults.

I have a config.yaml with parameters like so:
ACAPY_WALLET_SEED: Seed000000000000000000000000000
ACAPY_WALLET_TYPE: indy
ACAPY_WALLET_STORAGE_TYPE: postgres_storage
ACAPY_WALLET_STORAGE_CONFIG: '{"url":"<IP_ADDRESS>:5432","wallet_scheme":"DatabasePerWallet"}'
ACAPY_WALLET_STORAGE_CREDS: '{"account":"postgres","password":"mysecretpassword","admin_account":"postgres","admin_password":"mysecretpassword"}'
ACAPY_WALLET_NAME: verses_wallet
ACAPY_WALLET_KEY: abcd1234
ACAPY_GENESIS_URL: http://dev.greenlight.bcovrin.vonx.io/genesis
ACAPY_INBOUND_TRANSPORT: <IP_ADDRESS> 8020
ACAPY_OUTBOUND_TRANSPORT: http
ACAPY_ADMIN: <IP_ADDRESS> 8021
ACAPY_ENDPOINT: http://<IP_ADDRESS>:8020
ACAPY_ADMIN_INSECURE_MODE: true
ACAPY_LABEL: BaseAgent
ACAPY_PUBLIC_INVITES: true
ACAPY_MULTITENANT: true
ACAPY_MULTITENANT_ADMIN: true
ACAPY_JWT_SECRET: abcd1234

I keep the config file at the root level and run the start script:
 ./scripts/run_docker start --arg-file ./config.yaml

But no matter where I put it I always get the error:
aca-py start: error: File not found: ./config.yaml

I've tried pretty much every variation of path I can think of, ranging from just config.yaml to ../../../../config.yaml and putting it in various folders, including the Docker folder, the aries_cloudagent/config folder and anywhere else sensible, but it is never able to locate it.
Is there a specific location or argument that's required?

I assume you run aca-py inside docker right?
I had a similar issue providing the genesis-transaction-file.
When you use the docker Container, you start a new and clean ubuntu machine. Therefore your file is not inside the container.
To put it inside, you need to edit the Dockerfile which ./scripts/run_docker is starting.
there you need to Add your file.
Are you familiar with this? If not i will post an example of how to do that

That makes a lot of sense. If you would't mind posting an example snippet that I can adapt and add to the Dockerfile I would greatly appreciate it

Sure.

This means, your Dockerfile to be used is inside docker/Dockerfile.run.
Opening it (e.g. with vi, vim or any other texteditor):
FROM bcgovimages/von-image:py36-1.15-1
ENV ENABLE_PTVSD 0
ADD requirements*.txt ./
RUN pip3 install --no-cache-dir -r requirements.txt -r requirements.dev.txt
ADD aries_cloudagent ./aries_cloudagent
ADD bin ./bin
ADD README.md ./
ADD setup.py ./

RUN pip3 install --no-cache-dir -e .

RUN mkdir logs && chown -R indy:indy logs && chmod -R ug+rw logs

ENTRYPOINT ["/bin/bash", "-c", "aca-py \"$@\"", "--"]

Here you need to add your file like so:
ADD
for example:
ADD myYamlFile ./
To check, if all worked and if your file is inside the container, execute this command after your container has started (after you have run run_docker)
docker exec  ls
docker exec  cat
I hope this was helpfull

That now works perfectly, thank you so much.
However, now that it works it's uncovered some issues about the argument parsing itself:

Formatting the variables as the help section suggests e.g. ACAPY_SEED instead of --seed does not work, as it seems like the program is trying to interpret them as normal anyway:

aca-py start: error: unrecognized arguments: --ACAPY_WALLET_SEED=Seed0000000000000000000000000000

The variables only work if you present them in the normal format, so:
seed: Seed0000000000000000000000000000
wallet-type: indy
wallet-storage-type: postgres_storage
wallet-storage-config: '{"url":"<IP_ADDRESS>:5432","wallet_scheme":"DatabasePerWallet"}'
wallet-storage-creds: '{"account":"postgres","password":"mysecretpassword","admin_account":"postgres","admin_password":"mysecretpassword"}'
wallet-name: verses_wallet
wallet-key: abcd1234
genesis-url: http://dev.greenlight.bcovrin.vonx.io/genesis
inbound-transport:
  - [http, <IP_ADDRESS>, 8020]
outbound-transport: http
admin: [<IP_ADDRESS>, 8021]
endpoint: http://<IP_ADDRESS>:8020
admin-insecure-mode: true
label: BaseAgent
public-invites: true
multitenant: true
multitenant-admin: true
jwt-secret: abcd1234

Note that inbound-transport in particular has to be formatted in this specific way, or you get the error message:
aca-py start: error: argument -it/--inbound-transport: expected 3 arguments

I was only able to get it to work by copying the formatting found in the demo-args.yaml file found in /demo
I think that the args YAML instructions need to be updated to reflect what's required

Formatting the variables as the help section suggests e.g. ACAPY_SEED instead of --seed does not work, as it seems like the program is trying to interpret them as normal anyway:

aca-py start: error: unrecognized arguments: --ACAPY_WALLET_SEED=Seed0000000000000000000000000000

This is the name of the environment variable, not command-line input, so:
ACAPY_WALLET_SEED=Seed0000000000000000000000000000 ./scripts/rn_docker start ... etc

The variables only work if you present them in the normal format, so:
seed: Seed0000000000000000000000000000
wallet-type: indy
wallet-storage-type: postgres_storage
wallet-storage-config: '{"url":"<IP_ADDRESS>:5432","wallet_scheme":"DatabasePerWallet"}'
wallet-storage-creds: '{"account":"postgres","password":"mysecretpassword","admin_account":"postgres","admin_password":"mysecretpassword"}'
wallet-name: verses_wallet
wallet-key: abcd1234
genesis-url: http://dev.greenlight.bcovrin.vonx.io/genesis
inbound-transport:
  - [http, <IP_ADDRESS>, 8020]
outbound-transport: http
admin: [<IP_ADDRESS>, 8021]
endpoint: http://<IP_ADDRESS>:8020
admin-insecure-mode: true
label: BaseAgent
public-invites: true
multitenant: true
multitenant-admin: true
jwt-secret: abcd1234

Note that inbound-transport in particular has to be formatted in this specific way, or you get the error message:
aca-py start: error: argument -it/--inbound-transport: expected 3 arguments

I was only able to get it to work by copying the formatting found in the demo-args.yaml file found in /demo.
I think that the args YAML instructions need to be updated to reflect what's required

Every time the YAML is updated (even changing a single character), the entire image is rebuilt from scratch. Is there any way of putting in an ignore statement that doesn't rebuild just because the YAML changed? (Apologies if this isn't feasible, I'm far from a Docker expert)

Ok finally almost there, my final issue is that whilst trying to use environment variables, the formatting for multi argument variables is unclear, specifically for ACAPY_ADMIN and ACAPY_INBOUND_TRANSPORT
None of the following variants are accepted:
ACAPY_INBOUND_TRANSPORT: http <IP_ADDRESS> 8020
ACAPY_INBOUND_TRANSPORT: http, <IP_ADDRESS>, 8020
ACAPY_INBOUND_TRANSPORT: [http, <IP_ADDRESS>, 8020]
ACAPY_INBOUND_TRANSPORT: 'http <IP_ADDRESS> 8020'
ACAPY_INBOUND_TRANSPORT: http <IP_ADDRESS> 8020
ACAPY_INBOUND_TRANSPORT:
  - http
  - <IP_ADDRESS>
  - 8020
 ACAPY_INBOUND_TRANSPORT:
  - module: http
  - host: <IP_ADDRESS>
  - port: 8020

All are rejected with errors such as:
services.versesagent.environment.ACAPY_INBOUND_TRANSPORT contains ["http", "<IP_ADDRESS>", 8020], which is an invalid type, it should be a string, number, or a null

aca-py start: error: argument -it/--inbound-transport: expected 3 arguments

How should these multi-argument variables be formatted?

Hi @james-cohen , FYI we use configargparse (https://pypi.org/project/ConfigArgParse/) to handle these parameters.
I'm not sure offhand how to handle multi-value parameters in environment variables, but I can say from experience that these are not 100% supported.
We've typically used environment variables for parameters that we don't want to record anywhere (such as passwords and DID seeds) and for the rest we use either the aca-py command-line arguements or yml config files.

Good idea, I basically got a bit too obsessive with trying to make everything need and tidy but for those multi-parameter, non-secret ones I'll just pass as command line variables instead. Thanks!


User - Update password bug.

On the update API, if I give a password field, it's getting saved, and while login it's throwing an error as Invalid credentials.

Fixed the user Update API.
Merge request reference: #6


Class purpose and usage description in comments

Hi. I've read some of source codes in this projects and have a suggestion for its authors.
You did your best in commenting and describing class methods, properties, etc. But the main purpose and usage of class is not mentioned in file comments. On the other hand, as I think, all of classes and tools inside PHP Telegram Bot are not described in Wiki. So I think it's better to mention to class usages inside comments or completing Wiki (That is much better).

We are working on this


Having Trouble With Colors

When I import the *.terminal file into terminal preferences, all I see in the shell window is a light blue font over a gray background.
Do you know why none of the colors are working?

Hi,

Can you add screenshot?
Did you installed z-term?
Did you activated it in settings of terminal?

Using regular Terminal that comes with Mac. I did try the *.itermcolors profile in iTerm 2 with no luck though.

See #4 :)
Let me know if you're still running into issues!

Interesting that we need to create a .bash_profile to have ls colors etc. Should have colors by default in OS X.
Anyway, thanks Nathan that solved it!


Why does index_exists take an Index<'static>

Description
I am trying to dynamically query if an index exists or not using the following code snippet:
let myindex = get a &str here (tried it with clone() as well)
let response = match client.index_exists(index(myindex)).send() {
    Ok(x) => x,
    Err(x) => panic!("Could not send an Index Exists request.")
};

and I get the following error:
    |         let myindex = get a &str here (tried it with clone() as well)
    |                      ^^^^^^^^^^-------------------
    |                      |
    |                      borrowed value does not live long enough
    | } <- END OF PROGRAM
    | - `myindex` dropped here while still borrowed

Hi @DevQps :wave:
This is just for simplicity on the client side, but it isn't strictly necessary and I'd like to fix it up so you can use borrowed data (since we do eventually just build an owned url from it). In the meantime you can create an owned string to use:
let myindex = get a &str here (tried it with clone() as well)
let response = match client.index(myindex.to_owned()).exists().send() {
    Ok(x) => x,
    Err(x) => panic!("Could not send an Index Exists request.")
};

We've got #313 to track fixing this up so I'll close this one in favour of that issue. Please feel free to re-open though if you run into any more problems!


unused: 2.1 -> 2.2

Motivation for this change
Unused 2.2 just released: https://github.com/unused-code/unused/releases/tag/0.2.2
Things done

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

[ ] NixOS
[x] 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/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 127420 run on x86_64-linux 1

1 package built:

unused

LGTM


Prepare examples for Store and Node usage

I added example code and hints, hope its enough for the start.


[dev-v2.9] go get workflow is missing the go generate changes

forwardport: https://github.com/rancher/rancher/issues/47014

the PR to fix the CI is merged and it successfully created a go get PR for rke: https://github.com/rancher/rancher/pull/47405 hence closing this.


[Feature Request] optional plugin version manifest like package-lock.json or Gemfile.lock

Feature Request
I think it would be great if vim-plug optionally generated a plugin version manifest file (like npm's package-lock.json or rubygem's Gemfile.lock) that could be checked into version control. An additional feature would be optionally installing the versions directly from the lockfile (e.g. :PlugInstallFromLock). This would provide the following benefits:

Multiple machines could use the exact same versions of all plugins (I have my .vimrc with vim-plug installed on all machines, but plugin functionality often varies a little bit owing to different versions).
If there is a regression when doing a :PlugUpdate, the previous versions of all plugins can be determined (from the version-controlled lockfile) and plugins can be bisected to determine what change in what plugin caused the regression.
If the latest master of a plugin is broken, all other plugins could be rolled to the latest, while the bad plugin can be pinned to the last known stable version until there is an upstream fix.

@junegunn if you think this is a valid feature but don't have velocity to implement it I would be interested in writing a PR (but would probably need some assistance).
Proposed example lockfile in JSON (picked arbitrarily, another serialization format could be used):
{
  "plugLockVersion": 1,
  "plugins": [
    {
      "repo": "junegunn/fzf",
      "revision": "334a4fa15920b08f7be2432398e620587cc6f079"
    },
    {
      "repo": "junegunn/fzf.vim",
      "revision": "d4d36b412bdc53e8bdaf0e2714a2227f51781d68"
    }
  ]
}

Type:

[ ] Bug
[ ] Enhancement
[x] Feature Request
[ ] Question

OS:

[x] All/Other
[ ] Linux
[ ] OS X
[ ] Windows

Vim:

[ ] Terminal Vim
[ ] GVim
[x] Neovim

I just discovered the :PlugSnapshot command...

somewhat related https://github.com/junegunn/vim-plug/issues/906

I'm guessing my workflow for getting what I discribed in my OP but leveraging the existing :PlugSnapshot command would be something like:

Grab snapshot of current plugins with :PlugSnapshot $DOTFILES/vim-plug-snapshot.vim
Commit snapshot script to version control
On additional systems, pull latest snapshot script from version control, source in vim :source $DOTFILES/vim-plug-snapshot.vim
To update dependencies, :PlugUpdate then go to step 1 to create a new snapshot.
If there are regressions, bisect between the last two snapshots in version control.


parseUserData Support

Coming from @websanova/vue-auth, I was wondering if there is an equivalent configuration of parseUserData to allow me to extract/parse the user data from within my API response, for example when my response looks something like this:
{
  "status": "success",
  "data": {
    "username": "admin",
    "role": "admin"
  }
}

And the meaningful user data is only
{
  "username": "admin",
  "role": "admin"
}

I searched the docs and source code and couldn't find an alternative approach.

You have use keyUser. This option is available for all child options like login, refreshData....
const auth = createAuth({
    rolesKey: "type",
    notFoundRedirect: "/",
    fetchData: {
      enabled: true,
      cache: true,
      keyUser: "data"
    },
    refreshToken: {
      enabled: false,
    },

    plugins: {
      router,
    },
    drivers: {
      http: {
        request: app.config.globalProperties.$api,
      },
      auth: driverAuthBearer,
    },
  })

Thank you! I got it working as intended.
When checking the docs, I was wondering what the keyUser option is for, since the description for it and the userKey options are missing.
After reading the source code and testing, I understood that the userKey defines the name for the cookie or storage entry of the cached user data.
Also, I'm not sure if this still applies to this thread, but I have another instance where the user contains a boolean "is_admin" field instead of a role field. I tried to set the rolesKey to "is_admin" and use $auth.check(true) to check for admin access, unfortunately without success. A workaround would be to append the role: "admin" to the user object when fetching the user details, but I didn't find a way of doing it.

try setting the return from your api to include roles: ['admin'] then call .check('admin')

Thank you for your time. I would like to keep the api unchanged, to ensure compatibility between clients (old and new).
In @websanova/vue-auth there was a parseUserData configuration function that would allow changing the user object when fetching the user.
Just to be sure, there is no way to change the user object right after the fetch, to append the roles: ['admin'] is case of is_admin: true, is there?


How to change the  title icon ?

Is there anyway to change the default flet icon in the title bar. I have read the flet official website but nowhere found the any suggestion.
I am developing an application and almost completed. Struggling in changing title icon.
Any suggestion will be appreciated.
Thanks in advance.

It's currently not supported. In the future versions that will be done by rebuilding Flet viewer executable with custom media and metadata.

It's currently not supported. In the future versions that will be done by rebuilding Flet viewer executable with custom media and metadata.

hello. Is it supported now?


NoiseDerivs2D (3DBuffer).v4p

hello Kyle,
there's a bug in the v4p file formatting
https://github.com/everyoneishappy/InstanceNoodles/blob/master/nodes/modules/Compute/Value Buffer/NoiseDerivs2D (3D Buffer)/NoiseDerivs2D (3D Buffer).v4p
cheers

Would you mind giving this a try?  Tbh I've never used that node...
Should return the noise value in the X component and it's 2D derivatives in the Y & Z
https://www.dropbox.com/sh/o4ttne413hwz7e0/AAA2BOw9ZvljHDOogLDxhlkEa?dl=0

looks good,
just replaced the offfsetXY ioBox to a vector2D one and add a naive help patch!
NoiseDerivs2D (3D Buffer).zip


[Enhancement] Division

Is your feature request related to a problem? Please describe.
Currently, you can multiply, add and subtract in the skill editor. Divisions are not possible so a lot of math is unavailable, which can create frustrating situations.
For example, I've been trying to make a system that takes an attack speed value and turns it into a number to use for cooldowns. It is simply not possible as I would have to divide 1 by the value I have.
Describe the solution you'd like
A value divide mechanic that allows you to divide two values by eachother, a number by a value and a value by a number.
For example:
value/value
X/value
value/X
Would all be very useful, where X does not have to be a preexisting value and is a fixed number.
Describe alternatives you've considered
I have worked around with placeholders and placeholder math but I have currently hit a road block trying to get info that doesn't have a placeholder for it, whereas SAPI could simply read item lores and such.
Additional context
That is all.

In general it would be more useful to just have a "Value Math" function instead of the current system because skills can get bloated, but I'd be happy with just being able to do all the math that I need in the first place.

Accidentally closed, whoops

copy of https://github.com/promcteam/proskillapi/issues/311
additionally this too https://github.com/promcteam/proskillapi/issues/175


[🐞] MakeItYours's CopyCSSConfig shiki code flickers when clicking on "Copy CSS" button

Perhaps using a computed sig would solve it 🤔

Perhaps using a computed sig would solve it 🤔

How to reproduce?

Wait whaat. It doesn't happen on qwik-ui.com 😅. I thought it did! Maybe this was fixed by your recent shiki PR!!! Awesome either way. So closing 🙈


Allow predefined SMS messages and Emails to be sent via existing SMS/Email implementations

Allow SMS messages without SMS campaign to be saved

One use case that I see is sending one time passwords for two-factor authentication via SMS/Email.

@MegaAlex Currently provider id is fetched from message-gateway component while creating a sms campaign. So how does this provider id is configured with respect to the use case mentioned by you.

@nazeer1100126 In regards to two-factor authentication, I was thinking something along the lines of using global configurations or a configuration endpoint just for two-factor. Ultimately the goal of this PR is to make the SMS & Email more generic so they can be used not only for login email and sms campaigns.
As for the SMS service, setup of tenant & sms bridges in the message gateway directly from Fineract is part of my GSoC proposal but it should be discussed further.

PR is updated.


Corrected the spelling of JavaScript in languages.json and App.js

Fixed the spelling of "JavaScript" in languages.json and App.js


How can i animate from 1 to 0

Hello,
I want to animate the progress bar as a circle from a full circle to empty. Is it possible to do that with your API ?

https://jsfiddle.net/jh9dvme6/1/

it works


Graph

@stefuhnee,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


split json-schema/schema validation to its own repository

At the moment, configuration files and their validation both resides within siibra-python. This sometimes creates a cyclic dependency, and cause siibra-configuration/siibra-python to fail.
It could be the case that moving the siibra configuration json schema and its validators into a separate repository may fix this issue.

[ ] Create a public repo for all schemas in jugit
[ ] Add check schema to the new repo.
[ ] Remove schemas from siibra-python
[ ] Add functionality to inform deprecation of schemas


modify: use arrayList head value

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (11679e7) 74.10% compared to head (2c4ddab) 74.23%.

Files
Patch %
Lines

src/main/scala/domain/table/dml/RecordValue.scala
93.75%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #113      +/-   ##
==========================================
+ Coverage   74.10%   74.23%   +0.13%
==========================================
  Files          20       20
  Lines         390      392       +2
  Branches       30       43      +13
==========================================
+ Hits          289      291       +2
  Misses        101      101

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


SetWindowSizeCallback: user configurable callback

I wanted this, so I added it. Feel free to take it.

Hi @RandomErrorMessage, I prefer not exposing this callback. In any case, what was your specific use scenario to require it?

Just a preference to avoid having to do an additional window size check every frame, nothing special.


write to OSM

as a user I want to update healthsite.io attributes directly to OSM so that the data is available to others via openstreetmap
https://docs.google.com/spreadsheets/d/1ycaqJH-GHHs-Z5wyNpRaWOh8HWCZpSz_K-nz5a34imw/edit?usp=sharing
Notes:
Currently healthsites.io saves to a postgres database. We want to use openstreetmap as the database and postgress as a caching service to present qualitative analysis

v2 make it true


Add filters on the HBase side

The Scan object of HBase accepts a Filter object to avoid retrieving all the rows. This allows to

set a startkey or a endkey
filter rows based on a predicate of the rowkey
filter rows based on a predicate of the cell content

We should allow similar functionality. Notice that the startkey and endkey can be set on the TableInputFormat directly, without going through the Scan and Filter objects.
Since the possible filtering criteria are quite a lot, the issue of providing a simple API arises. The possibilities are:

allow passing a Filter object of HBase directly
adding more optional parameters to the read API
provide users with an object similar to Filter, but custom to this library.

The first option seems undesirable, both to avoid exposing a configuration parameter over which we have no control and because Filter is entangled with Java HBase API.
There exists a fourth option, that would probably be neat: return an RDD which wraps the one that we are returning right now and proxies transparently all methods (such as map etc.) but filter. The filter method would instead be implemented by returning a new RDD with the appropriate Filter object applied.
Unfortunately, this would work for simple predicate, but would run into all sorts of issues when the predicate depends on libraries or resources which may not be available on the HBase server.
Once the desired API is fleshed out, adding the filter should be a simple matter

Scan filtering is now available in version 0.5.0.
However we would like to investigate if other approaches that @andreaferretti describes are feasible, such that Java HBase client API are completly hidden to the user.


isKey problem

Hello Guys,
I have a question below.
Edited.
I got the following warning:
warning.js:36 Warning: flattenChildren(...): Encountered two children with the same key, 1010111. Child keys must be unique; when two children share a key, only the first child will be used.
Is there anyway that I can hide this in console logs?
It is doing the job for me at least.
Please check.

hello, would you please show me your example? and you mean you have duplicated key in your data?

Hello!
I already fixed this problem.
Thanks!


Hoster location error

Subject: Hoster location error
Description:
I searched for hosters in Dundee Scotland and ...
Results:
https://couchers.org/user/adam847 came up despite being in Iowa City, Iowa. Seems odd!
surely they should be in iowa not dundee?
Backend version: develop-7c33e18d
Frontend version:
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Page: https://couchers.org/search?location=City+Centre%2C+Dundee%2C+Scotland%2C+United+Kingdom&lat=56.4567487&lng=-2.9705355
User: zipnun_mendak (17012)

data issue with that user


implement /v1/search in the V2 registry API

Registry API v1 had /v1/search but there is nothing similar in the V2 API.
This is quite important for private registries.
Please port the v1 behavior to V2, thanks!

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO


Exit menu on windows is not working properly.

In your compiled latest releases any of menu items on exit section does 1 exact thing - simply exits ES. Shutdown system and restart doesnt work.

Code in platform.cpp looks ok, except id change #ifdef WIN32 to #ifdef _WIN32, cause _WIN32 is defined by the compiler, not the environment.

i have this issue as well on windows 10. ive taken the raw command and compiled it by itself and works just fine. so if anyone figures out where the disconnect is id be grateful as well.

fix GuiMenu.cpp line 503 ,add this code:
#ifdef WIN32
runRestartCommand();
#endif

Pull request: https://github.com/jrassa/EmulationStation/pull/22
#22


[FEA] Support percentile function

It would be better to support percentile function:
Codes:
spark.sql("SELECT percentile(col, 0.3) FROM VALUES (0), (10) AS tab(col)").show

driverlogs:
! <Percentile> percentile(col#28, 0.3, 1, 0, 0) cannot run on GPU because GPU does not currently support the operator class org.apache.spark.sql.catalyst.expressions.aggregate.Percentile

Duplicate of https://github.com/NVIDIA/spark-rapids/issues/6266


allow for dark background in the debugger view

I prefer a darker environment when coding, and having the Chrome window be dark except the part that I cannot hide, is making my eyes hurt. This is for the people that prefer the darker color scheme when developing.

i dont see why the tests are failing.. its not from my changes i think?

Can you add some screenshots that show the new dark mode as well as the settings window with the checkmark? Also, can you provide a test plan, e.g. How did you test this?

first one is as it is now (plus the checkbox),  and the next one is the new dark one.

well, i dont know how to test this.. its just changing the css. does this screen have any other tests? maybe i could get some idea from that, if there is anything.

Test plan doesn't necessarily require adding tests. When you open a new PR, there's a template with all the info we need, including a test plan describing how this change was tested.

ok, yes. well, as i wrote, i wanted a darker environment, and i made a simple one. then i tested by actually clicking in the checkbox to make sure its working. i used it for a whole day and it felt much easier on the eyes.

Thanks for adding the test plan (screenshots). Merging.
@facebook-github-bot shipit

@facebook-github-bot shipit

i will rebase it and see if that helps..


Extend support of :filter

It lets one filter output of a few more commands:

:function
:command
:let
:mark
:jumps
~:clist and llist (only by file name or module name.~

Includes tests.
This extends the closed PR: #2327.

Codecov Report

Merging #2856 into master will increase coverage by 0.04%.
The diff coverage is 90.9%.

@@            Coverage Diff             @@
##           master    #2856      +/-   ##
==========================================
+ Coverage   76.59%   76.63%   +0.04%
==========================================
  Files          93       93
  Lines      137098   137109      +11
==========================================
+ Hits       105008   105075      +67
+ Misses      32090    32034      -56

Impacted Files
Coverage Δ

src/option.c
85.15% <100%> (+0.07%)
:arrow_up:

src/mark.c
83.61% <100%> (+1.92%)
:arrow_up:

src/syntax.c
78.38% <100%> (+0.01%)
:arrow_up:

src/eval.c
83.36% <100%> (+0.9%)
:arrow_up:

src/userfunc.c
87.5% <50%> (+0.18%)
:arrow_up:

src/gui_gtk_x11.c
47.52% <0%> (-0.45%)
:arrow_down:

src/window.c
81.87% <0%> (-0.04%)
:arrow_down:

src/if_py_both.h
76.31% <0%> (ø)
:arrow_up:

src/os_unix.c
57.21% <0%> (+0.13%)
:arrow_up:

... and 4 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 21f990e...998c46b. Read the comment docs.

@brammool there's a item in todo file that should be updated: the extended :filter support is in this PR.

@brammool is there a chance to include this? It's hanging here for a while already....

I think this is very useful. But is there any update for documents?
No one knows when the todo item will be merged. In a few days, weeks, months, or years...

@k-takata I added documentation; as I recall @brammool didn't want to be explicit in what things are supported, hence the style of the doc.


failed to find any PEM data in certificate input  proxy exited with code.

After updating to 5.2.3 version, I´ve got the error "failed to find any PEM data in certificate input
proxy exited with code".

Would you be able to post a screenshot or a more full stack trace?