Default file template incorrectly uses trim command

Describe the bug
The default file template uses "trim" has a block identifier, which is not valid in Nunjucks. It is only a filter. You can see this by attempting to copy the default template directly into the plugin's settings field for customization and it will not accept the default as written. Removing the {% trim %} and {% endtrim %}` allows the default template to be usable.
PS - it sure would be nice to have the default template to just be actual default editable text. Right now, as soon as you make modifications in the plugin settings, it erases the default value - causing users to have to completely reconstruct the template from scratch.
Method of syncing

N/A

To Reproduce

Go to plugin settings, type in the default file template and watch it turn red when you type {% trim %}

Expected behaviour
The default template text should not have a trim block.
Screenshots

N/A

Metadata

OS: Windows
Plugin version: latest
Obsidian version: latest

UPDATE: I notice now that you enabled a trim extension (very handy) for trimming out empty lines. Unfortunately, the validation routine for the file template does not support it, and thus the default template (or any modification that uses the trim and endtrim extensions will be rejected.
PPS - Thanks again for such a marvelous plugin!

I also wanted to extend the default template and found I couldn't. I fetched it from source, but as the OP points out, the trim function is not valid according the validation routine.
I've resorted to patching the source code, but this won't survive updates.

I've just released as of this morning version 1.8 of the plugin. It has a completely redesigned template editor that is meant to make it much easier for you to modify templates:

You can revert to default plugin template with a click of a button
You can override the default plugin with a click and modify it

I've tried recreating the issues you were all experiencing with this version and I am not able to create the errors.
Sure, adding a {% trim %} block on its own will immediately error. However, once you add the closing block statement {% endtrim %} the template is valid and the plugin accepts it.
Give it another try please and let me know if that is not the case.
A video screenshot of the problem would help me a lot get to the bottom of this problem faster.

Haha... thanks for confirming.
But... as they say, a picture is worth a thousand words.
I now understand the issue. You are using the custom trim block that is only configured to be used on the file template and NOT the highlights template.
The easiest solution is to expose this in the highlight template.
I do need to think about this. I am actually not a big fan that all users are exposed to this trim block that I never intended to be used by users. It is non-standard and it is hard for future users to get their head around it. I am wondering if its possible that the plugin auto-truncates all empty lines by default.

If it makes your life any easier -- I don't actually need to the trim function in highlights. But it wasn't working in the main template before your latest update -- I took the original template and pasted it into the box and it wouldn't accept the {% trim %}{% endtrim %} tags, but it doesn't seem to be an issue since your latest update.


npx create-react-app my-app throws error

When trying to create a react-app using this command the entire process abords in middle
Node Version: v8.9.4
NPM Version: 5.6.0
here is the error
npm ERR! code EINTEGRITY
npm ERR! errno EINTEGRITY
npm ERR! Invalid response body while trying to fetch https://registry.npmjs.org/convert-source-map: Integrity verification failed for sha512-J6GnkARaHVLFHhnrlZIiRnkqL+3foACS3obxtO6zGanFfm6ryWhL26INoCboxH0fyqjvgdbrj+OyzHWmoAsQWQ== (C:\Users\dhunm\AppData\Roaming\npm-cache\_cacache\content-v2\sha512\27\a1\a790045a1d52c51e19eb95922246792a2feddfa00092de86f1b4eeb319a9c57e6eabc9684bdba20da026e8c47d1fcaa8ef81d6eb8fe3b2cc75a6a00b1059)

npm ERR!      C:\Users\dhunm\AppData\Roaming\npm-cache\_logs\2018-04-12T19_30_21_906Z-debug.log

I assume it was an npm issue. I use this command on a weekly basis and have no problems at all.

I reinstalled node and I still have the same issue, I am not an expert in debugging these types of issues what should I do.

I am facing the same problem trying to run npm install -g create-react-app I think there is some issue with the create-react-app package itself.

This issue got solved, I followed these steps.

Update 14.06 (3/3): after deleting NPM cache (with npm cache clear --force) and /node_modules from all target systems (RHEL6, OSX, Windows 8), reinstalling the project modules with npm install --no-shrinkwrap --update-binary and shrinkwrapping afterwards (with npm shrinkwrap), I got an identical npm-shrinkwrap.json in every system, that consistently builds across all of the RHEL/OSX/Windows8 systems and does not report SHA checksum issues anymore. It is beyond my understanding how it solved the issue.

Seems to be a network issue. Try changing the internet source, switch to some other wifi. Worked for me. I saw this answer here https://github.com/facebook/create-react-app/issues/3057.


Values in json_schema_extra get sorted automatically during model_json_schema

[x] This is an actually a bug report.
[ ] I am not getting good LLM Results
[ ] I have tried asking for help in the community on discord or discussions and have not received a response.
[ ] I have tried searching the documentation and have not found an answer.

What Model are you using?

[ ] gpt-3.5-turbo
[ ] gpt-4-turbo
[ ] gpt-4
[ ] Other (please specify)

Describe the bug
This isn't a bug with instructor per se but any examples you add to json_schema_extra get sorted: https://github.com/pydantic/pydantic/issues/7580. This can impact the quality of the generation if you have "thought" for instance to trigger CoT, it will get output at the end and then steer the model towards doing that too.
There should be an option to avoid this in pydantic but thought it was worth noting in the example since this threw me off.
To Reproduce
Steps to reproduce the behavior, including code snippets of the model and the input data and openai response.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

i've always used chain_of_thought and never noticed, will talk to the pydantic team

I also was putting user as a key to simulate the user request and then
the response. The QA example didn’t seem practical because then it requires
the user prompt in the model? Am I missing something there?
On Sat, Mar 16, 2024 at 5:05 PM Jason Liu @.***> wrote:

i've always used chain_of_thought and never noticed, will talk to the
pydantic team
—
Reply to this email directly, view it on GitHub
https://github.com/jxnl/instructor/issues/508#issuecomment-2002195409,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFMAUL54ZUIE4GC2NULGXLYYTM3ZAVCNFSM6AAAAABEZSVOJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBSGE4TKNBQHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

can you give a code snippet? i can't see what youlre thinking about

Something like this:
class NextAction(BaseModel):
    thought: str = Field(
        description="Think about the best next action based on what the user said."
    )
    action: Literal["laugh", "cry"]

    model_config = ConfigDict(
        json_schema_extra={
            "examples": [
                {
                    "user": "a funny joke",
                    "thought": "jokes are funny, i should laugh",
                    "action": "laugh",
                },
                {
                    "user": "a sad story",
                    "thought": "the story is sad, i should cry",
                    "action": "cry",
                },
            ]
        }
    )

I ended up going with something like this:
class NextAction(BaseModel):
    thought: str = Field(
        description="Think about the best next action based on what the user said."
    )
    action: Literal["laugh", "cry"]

    model_config = ConfigDict(
        json_schema_extra={
            "examples": [
                {
                    "user": "a funny joke",
                    "thought": "jokes are funny, i should laugh",
                    "action": "laugh",
                },
                {
                    "user": "a sad story",
                    "thought": "the story is sad, i should cry",
                    "action": "cry",
                },
            ],
            "examples_description": "The `user` in examples is only for illustrative purposes, you should not include it in your output."
        }
    )

which seems to work well.
this is the sorting issue I was talking about:
>>> print(json.dumps(NextAction.model_json_schema(), indent=2))
{
  "examples": [
    {
      "action": "laugh",
      "thought": "jokes are funny, i should laugh",
      "user": "a funny joke"
    },
    {
      "action": "cry",
      "thought": "the story is sad, i should cry",
      "user": "a sad story"
    }
  ],
  "properties": {
    "thought": {
      "description": "Think about the best next action based on what the user said.",
      "title": "Thought",
      "type": "string"
    },
    "action": {
      "enum": [
        "laugh",
        "cry"
      ],
      "title": "Action",
      "type": "string"
    }
  },
  "required": [
    "thought",
    "action"
  ],
  "title": "NextAction",
  "type": "object"
}

this breaks CoT because the model thinks it' should output the action key first, which isn't what we want.
In debugging all of this, I also noticed that the schema that gets sent includes references for everything. For simple schemas where you're not even re-using the models this increases the token count + makes the usage more unreliable from what I can tell.
ie:
class NestedModel(BaseModel):
    note: str

class NextAction(BaseModel):
    thought: str = Field(
        description="Think about the best next action based on what the user said."
    )
    action: Literal["laugh", "cry"]
    nested: NestedModel

    model_config = ConfigDict(
        json_schema_extra={
            "examples": [
                {
                    "user": "a funny joke",
                    "thought": "jokes are funny, i should laugh",
                    "action": "laugh",
                },
                {
                    "user": "a sad story",
                    "thought": "the story is sad, i should cry",
                    "action": "cry",
                },
            ],
            "examples_description": "The `user` in examples is only for illustrative purposes, you should not include it in your output.",
        }
    )

turns into:
{
  "$defs": {
    "NestedModel": {
      "properties": {
        "note": {
          "title": "Note",
          "type": "string"
        }
      },
      "required": [
        "note"
      ],
      "title": "NestedModel",
      "type": "object"
    }
  },
  "examples": [
    {
      "action": "laugh",
      "thought": "jokes are funny, i should laugh",
      "user": "a funny joke"
    },
    {
      "action": "cry",
      "thought": "the story is sad, i should cry",
      "user": "a sad story"
    }
  ],
  "examples_description": "The `user` in examples is only for illustrative purposes, you should not include it in your output.",
  "properties": {
    "thought": {
      "description": "Think about the best next action based on what the user said.",
      "title": "Thought",
      "type": "string"
    },
    "action": {
      "enum": [
        "laugh",
        "cry"
      ],
      "title": "Action",
      "type": "string"
    },
    "nested": {
      "$ref": "#/$defs/NestedModel"
    }
  },
  "required": [
    "thought",
    "action",
    "nested"
  ],
  "title": "NextAction",
  "type": "object"
}

the LLM has to parse the references correctly along with do the task which I was getting poor results with.
I worked around the sorting + this referencing issue with this:
class DereferencedJsonSchema(BaseModel):

    @classmethod
    def model_json_schema(cls, *args, **kwargs):
        """Custom model_json_schema method to make it easier for an LLM to understand the schema.

        By default, pydantic will create refs stored within `$defs` for any
        schema. This is usually what you want but it unnecessarily complicates
        the schemas being sent to the LLM. If your not referencing a schema
        multiple times, this will also increase the token count. Additionally,
        pydantic will also sort keys. If you provide examples to
        `json_schema_extra`, pydantic will sort the keys within those examples
        which can negatively influence how the model generates it's output.

        We work around both those issues with this function and have seen much
        better results when using it.

        """
        json_schema = super(DereferencedJsonSchema, cls).model_json_schema(
            *args, **kwargs
        )
        json_schema_extra = cls.model_config.get("json_schema_extra")

        if json_schema_extra:
            # re-apply the extras to get around sorting
            json_schema.update(json_schema_extra)  # type: ignore

        # dereference the schema
        dereferenced: dict[str, Any] = jsonref.replace_refs(json_schema, proxies=False)  # type: ignore

        # remove defs
        dereferenced.pop("$defs")
        return dereferenced

which results in:
{
  "examples": [
    {
      "user": "a funny joke",
      "thought": "jokes are funny, i should laugh",
      "action": "laugh"
    },
    {
      "user": "a sad story",
      "thought": "the story is sad, i should cry",
      "action": "cry"
    }
  ],
  "examples_description": "The `user` in examples is only for illustrative purposes, you should not include it in your output.",
  "properties": {
    "thought": {
      "description": "Think about the best next action based on what the user said.",
      "title": "Thought",
      "type": "string"
    },
    "action": {
      "enum": [
        "laugh",
        "cry"
      ],
      "title": "Action",
      "type": "string"
    },
    "nested": {
      "properties": {
        "note": {
          "title": "Note",
          "type": "string"
        }
      },
      "required": [
        "note"
      ],
      "title": "NestedModel",
      "type": "object"
    }
  },
  "required": [
    "thought",
    "action",
    "nested"
  ],
  "title": "NextAction",
  "type": "object"
}

depends on the schema and how often you're referencing the same class, but I got significantly better results with this.

Could you tell me whwt jsonref is?


Error messages for super users

This fixes #334

I don't know if I can approve this. It's too concise and neat. Where is the drama, the thrill of confusion. I want to feel like a kid lost in Walmart, a man in the depths of the rainforest w/o as much as a match....
J/K
Well done :shipit:

LMAO!
You got me Tek, you got me!
Thanks for the laugh!
Kareem
On Jun 17, 2016 7:49 PM, "tekHudson"<EMAIL_ADDRESS>wrote:

I don't know if I can approve this. It's too concise and neat. Where is
the drama, the thrill of confusion. I want to feel like a kid lost in
Walmart, a man in the depths of the rainforest w/o as much as a match....
J/K
Well done [image: :shipit:]
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/on-site/StockAid/pull/350#issuecomment-226906581, or mute
the thread
https://github.com/notifications/unsubscribe/AGIDdvm_Zk-TzrhvQ0xp2HMmaVlfGQnJks5qMzKXgaJpZM4I40CR
.


Support managing image sprites in the embedded gl-style editor

[ ] Alter schema to support globally available sprites

[ ] Alter sprites table
[ ] Add query support
[ ] Add categorization and keyword support to organize sprites
[ ] Add methods for promoting sprites to public availability

[ ] Identify sprite urls and show image "decoration"
[ ] Show error if sprite url is invalid
[ ] Sprite picker component

[ ] Add ability to select from existing project sprites
[ ] Show publicly available sprites, organized by category
[ ] Search function
[ ] Add ability to upload new sprites
[ ] Ability to promote sprites as public for superusers


Provide a way to get fresh transient services without HostContainer

Currently, the only way to get fresh instances of a transient service are by using HostContainer.
It feels nasty to recommend using what is essentially an escape hatch instead of a more sensible API.
import { Service, HostContainer } from '@typed-inject/inject';
import { TimerService } from './timer.service';

@Service([
    HostContainer()
])
export class PageService {
    constructor (private container: ContainerInstance) { }

    async renderPage () {
        const timer = this.container.get(TimerService);
        timer.setName('page-render');
        timer.start();
        // Perform page rendering logic...
        timer.end();
    }
}

N.B. For any interested parties: it's considered an "escape hatch" because it's essentially an implementation of the service locator pattern: you pass in a container to your service, and the service requests values for it.  This makes a real headache for extending services, as they have to figure out what services that service requires from the implementation code.


Rollouts aren't evenly distributed

The algorithm implemented in user_in_percentage? is flawed, because it doesn't distribute evenly.
(1..1_000).select {|x| Zlib.crc32(x.to_s) % 100 < 25 }.count # => 271. Expected value would be 250

As a side note, CRC32 is prone to hash collisions (see here).

Doing a < or > vs an appropriate boundary in the 32 bit integer space should be an improvement over the current situation:

it would address the auto increment increment
while it would not provide 100% uniformness given the 32 bit space I'd say it's as close as is practical
still a single operation

Unless I'm missing something.

Let's close this issue as https://github.com/fetlife/rollout/pull/120 improved this a bit and relative error is small enough.


Add 1.46 role

1.46 will be replaced by 1.48
Do not merge

Aside from the help text that needs to change in the CSV, this looks good.
But can you confirm that the roles/v1.46 is simply a copy of the current roles/default? That's what it should be (I just want to make sure you didn't copy roles/v1.36 - the new version should be a copy of the roles/default directory).
I also +1 the changes you made to add the new 1.46 to the default supported images. That is all correct.

I am 100% sure that I copied the 1.36 and not the default 🤣 let me change it 😊

I think @jmazzitelli that if we are going to update this later to the version 1.48 we can avoid this merge and only do the PR to the branch 1.46 https://github.com/kiali/kiali-operator/pull/488 . What do you think?

only do the PR to the branch 1.46 #488 . What do you think?

yes, that's a good idea.

Closed in favor https://github.com/kiali/kiali-operator/pull/488


docs: a few usage examples for div

Adds a few usage examples for div operations. ref issue #283.
Additionally modifies:
pub const fn wrapping_rem

into:
pub const fn wrapping_rem_vartime

since it uses rem_vartime internally.

I just want to reiterate my position (last time, I promise) that usage examples for simple arithmetic operations like these are entirely excessive.

I just want to reiterate my position (last time, I promise) that usage examples for simple arithmetic operations like these are entirely excessive.

wrapping_rem_vartime and rem2k_vartime examples might be excessive, if you would like those removed, Im not opposed to it. The others did take me considerable time to get working without examples, it would be nice to leave them if possible.


Graceful shutdown

I'd like to explore ideas in how to gracefully shutdown a running http server. Now that go 1.7 supports context, we can use the request context to signal to all running requests the intent to shutdown the server and finish up their work because we're going to be exiting the program.
Initially the thought was to have a single parent server context where each routing context was derived, and if the parent was shutting down, cancelling the server context would signal to stop each child context. The problem here, is cancelling isn't the right thing to do, and it's actually the only option thats built into context.Context itself (see: https://github.com/golang/go/issues/14660#issuecomment-209028114).
On second thought, we could include a ShutdownCh on the request context, that would enhance each request to signal whenever the server is intending to finish up its work and stop processing. This is the best idea I have so far.

it's also likely someone will solve this outside of chi using Go 1.7's request context and it will "just work" with chi

We should discuss the Graceful Shutdown workflow:
On SIGINT/SIGTERM (or any other registered signal):
1a. Close listening socket and refuse any incoming connections.
1b. or .. Start responding with HTTP 503.
2. Disable keep-alive connections (how about http2?).
3. Start time.After(softTimeout) to let active requests finish.
4. When softTimeout expires, close the root context. Start time.After(hardTimeout).
5. Wait for all active requests to finish (middleware adding/removing value from channel or atomic.Uint64)
6. When hardTimeout expires, exit program.

softTimeout lets active requests finish their work without closing the context chain

I think all of the above except for 1a and 2 can be solved by middleware.

"1a" variant is much better (allows to use SO_REUSEPORT and start second instance)
P.S.: it's possible just to use https://github.com/tylerb/graceful (or get some inspiration there)

Yep, right now we're using goji/graceful, and it appears to work, so I'm curious what @pkieltyka is thinking re: his comment that what we're doing wouldn't work. Maybe subtle race conditions?

@cyx btw, this is what I was thinking https://github.com/goware/valve/blob/master/_example/main.go
its agnostic to chi, but of course works well with it

@pkieltyka nice, I see what you mean now -- you're very precise in the teardown process so I guess the way we're doing it now will be hand-waivy about things at best.
Excited to see something first class in chi?

@cyx yea exactly, it's important to preempt certain code blocks properly and that is where the valve package comes in. I'm going to update it's README soon, but the thought is its designed as an extension to context.Context, that adds an additional valve signal that when stopped will tell non-preemptive code blocks to stop processing and finish up, aka, the taps are turning off, stop what you’re doing.
it's already first-class to chi since it just relies on context.Context. It's not specific to just http servers, its useful for any kind. I have however moved the valve package to https://github.com/pressly/valve
I thought to write something like github.com/tylerb/graceful from scratch and having it right as a part of chi/graceful - but there is no point, I'd just be ripping their code for what does a very fine job. The missing part was signaling the shutdown and having proper code to let non-preemptable code finish. At this present time, I recommend the approach https://github.com/pressly/chi/blob/master/_examples/graceful/main.go but I'm always open to feedback and future ideas. Let me know how it works for all :)


Add comment action

Based on main.yml from course but intended to work on comments.

Test comment


Progress Overview on profile page is broken

It gives both an incorrect count of problems in each category (it only includes problems visible to the user instead of all problems in the category) and sometimes gives incorrect solved counts.

Thanks for reporting this; I'm working on fixing it up.
It's probably worth discussing whether revealing to total number of problems in each category is a bad idea or not. I imagine it may depend on what competition is being run.


Y axis values overlap scale label if values are float

Minor issue, but if the values on the Y axis are float (0.5, 1.0 etc) and the chart has a scale label, the numbers sort of overlap it:

An option to change the scale label positioning would be a good way to solve this.

Interesting that this happening. My likely guess is that the axis is measured and integer labels are assumed and then labels are changed and the axis is not re-measured. I recall fixing something similar a while ago. Not sure why it's returned.

This is the same as #4441.  That started without a scale label, and truncates the tick labels.  Noted there that adding a dummy scale label added enough space to prevent the truncation, but that a real label would overlap.  Bet the same workaround will work here.  Specify the step size as 0.5 instead of letting it default.  That will properly calculate the tick label extents.

Thanks @mMerlin! Closing as a duplicate of #4441


Check additional names from certificate alt names

It could be nice to add option to request https certificate from already found subdomains and check if any of them has additional subject alt names.
I did only fast scroll through code so sorry if it already does this

Hi @lafriks and thanks for opening this issue. You do have a point and i'll look into it. At the moment DELATOR only checks for certificate common names. I need to study google's certificate-transparency-go certificate types a little closer but i do believe that alt names should be retrievable with the following:
altNames := parsedEntry.X509Cert.Subject.ExtraNames
Don't hold me to account just yet though, i still need to do more research. Stay tuned.


Some fix for configure script of s390x.

What type of PR is this?
/kind bug
/kind failing-test
What does does this PR do / why we need it:
Add same fix with #4206.
Add some fix for importing java imagestream.
Which issue(s) this PR fixes:
Fixes #?
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:

/area system-PZ

@zhengxiaomei123 integration tests failed.

/retest

@prietyc123 @mohammedzee1000 this PR is modifying the cluster configuration script for s390x architecture. But the 4.6 tests that are failing are being run on x86, right? What I'm observing in the test failures is that odo catalog list services command is failing in Operator Hub integration tests because it can't find any Operators installed on the cluster.
In this PR, I'm seeing that KUBEADMIN_PASSWORD value is being removed from configuration script of s390x: https://github.com/openshift/odo/pull/4317/files#diff-e24df2f816e33c2abe791ad856ac6f922a218705363d4f3176ec5e12050b5cb2L103-L106. But why would that cause failures in our integration tests that are running on x86? That should use configure-installer-tests-cluster.sh script, no?

Interesting, that should not affect x86 indeed,

Interesting, that should not affect x86 indeed,

@zhengxiaomei123 also wondering why the sleep 5 is needed.

@zhengxiaomei123 also wondering why the sleep 5 is needed.

Failing due to https://github.com/openshift/odo/issues/4301 .
/retest

Failing due to https://github.com/openshift/odo/issues/4301 .
/retest

/lgtm

/lgtm

/lgtm

/lgtm

/retest

/retest

/lgtm

/lgtm

/approve

/approve

/approve

/approve


feat: add fix and test for asset handling of imported images

This a test case to fix #498

@dai-shi can you look at the failed typescript error - it is not shown when I run pnpm test local but I think it is related to the fact that I have changed the App component to an async function in ./e2e/fixtures/ssr-target-bundle/src/components/App.tsx - should I disable the error in entries.tsx?

Type 'Promise<typeof import("/waku/e2e/fixtures/ssr-target-bundle/src/components/App", { with: { "resolution-mode": "import" } })>' is not assignable to type 'Promise<{ default: ComponentType; }>'.

pnpm test shows the error for me.


API Requirement of Statically Generated Dynamic Applications

What problem does this feature solve?
Users would test their dynamic nuxt applications without the need of a public API of their data.
What does the proposed changes look like?
When "nuxt generate" command is used with any page component with asyncData that fetches data for page content, although replicates the original application, created static application still needs an API for client side page renderings. I am using generate command from time to time to upload my app to any static host (surge in my case) to show it to my friends. It would be really nice to disable client side routing on nuxt config file in order to have all links on the pages completely server rendered. For now, I need to click "open in new tab" for all pages.

This feature request is available on Nuxt community (#c9229)

See https://github.com/nuxt/rfcs/issues/22


Wifi and Bluetooth not working

Your EFI makes me surprised, everything works very well but except wifi and Bluetooth, i hope you will fix it soon, i am in Ventura.
Thank you

Thanks! But I am not the developer of the Wifi and Bluetooth drivers, support for these drivers stopped in macOS Monterey version, so I disabled these drivers on my EFI for macOS Monterey or newer. Here is the support forum link for these drivers:
IO80211Family modif

It looks like your Wifi and Bluetooth adapter is not the same as what I defined in the EFI specs, can you check what your Wfi and Bluetooth adapter is?

It looks like your Wifi and Bluetooth adapter is not the same as what I defined in the EFI specs, can you check what your Wfi and Bluetooth adapter is?

Yes, sure. I see in the device manager, my built in wifi is Intel(R) Dual Band Wireless-AC 3165 and the Bluetooth is Intel(R) Wireless Bluetooth(R). If it's what you want to know. Could you tell me how to get that information? Thanks

Please test this EFI folder: EFI.zip, I changed Wifi and Bluetooth drivers. After update your EFI folder, restart and install this app on mac: HeliPort.dmg, use this app to manage your Wifi adaper.

Please test this EFI folder: EFI.zip, I changed Wifi and Bluetooth drivers. After update your EFI folder, restart and install this app on mac: HeliPort.dmg, use this app to manage your Wifi adaper.

Thanks for your works, the wifi works fine, the bluetooth could connect to another devices too, but it can't not turn off bluetooth. And I don't know why after sign in Apple Account, the system will restart accidentally but it doesn't signed in. And I wonder if I could use this EFI file for Ventura?

Thanks for your works, the wifi works fine, the bluetooth could connect to another devices too, but it can't not turn off bluetooth. And I don't know why after sign in Apple Account, the system will restart accidentally but it doesn't signed in. And I wonder if I could use this EFI file for Ventura?

Yes you can use this folder with Ventura. About the Apple account, I set generic serials in the EFI and you'ill need to change the serials (MLB, SystemSerialNumber & SystemUUID) in the SMBIOS with new ones for MacBookPro14,1 for correctly work with Apple servers, you can generate with: GenSMBIOS, if you need help to generate, i can generate new ones and email you the EFI folder.

Thanks for your works, the wifi works fine, the bluetooth could connect to another devices too, but it can't not turn off bluetooth. And I don't know why after sign in Apple Account, the system will restart accidentally but it doesn't signed in. And I wonder if I could use this EFI file for Ventura?

Yes you can use this folder with Ventura. About the Apple account, I set generic serials in the EFI and you'ill need to change the serials (MLB, SystemSerialNumber & SystemUUID) in the SMBIOS with new ones for MacBookPro14,1 for correctly work with Apple servers, you can generate with: GenSMBIOS, if you need help to generate, I can generate new ones and email you the EFI folder.

Yes, please. i don't have any experience about this. It's nice if i could receive that EFI from you, you could put link here directly or send via email<EMAIL_ADDRESS>Many thanks again. Btw, i think the built-in sound works fine both input and ouput  but if you try to plug a headphone, the micro will doesn't work although the output sound works fine. It's great if you could find the root cause then fix it. I had searched in many internet source but change the layout id doesn't fix that.

Yes, please. i don't have any experience about this. It's nice if i could receive that EFI from you, you could put link here directly or send via email<EMAIL_ADDRESS>Many thanks again. Btw, i think the built-in sound works fine both input and ouput but if you try to plug a headphone, the micro will doesn't work although the output sound works fine. It's great if you could find the root cause then fix it. I had searched in many internet source but change the layout id doesn't fix that.

I sent the EFI folder with the new serials to your email. About the microphone, check if the mac changed the microphone when it was connected, and try to disable the ambient noise reduction, in the sound settings.

Yes, please. i don't have any experience about this. It's nice if i could receive that EFI from you, you could put link here directly or send via email<EMAIL_ADDRESS>Many thanks again. Btw, i think the built-in sound works fine both input and ouput but if you try to plug a headphone, the micro will doesn't work although the output sound works fine. It's great if you could find the root cause then fix it. I had searched in many internet source but change the layout id doesn't fix that.

I sent the EFI folder with the new serials to your email. About the microphone, check if the mac changed the microphone when it was connected, and try to disable the ambient noise reduction, in the sound settings.

Thanks for the file. I used it in Ventura but it doesn't work, touchpad, wifi, sound ... seems doesn't work. I will try it on BigSur then inform you later


Remaining features for Prototype

Recover, update, delete account
Users can send instant messages to each other
Update, delete realms
Moderator can add/remove moderators with equal or less access
Edit, delete topic, show time of last edit
Vote for topic - up/down, for "down" you need to give a reason from a list (spam, repost, not constructive, etc)
List topics - by  author - sort by: score, newest, controversial, top(24h, etc) - search by: title
A comment of arbitrary length  can be added to a topic, or to another comment.

User and Realm URLs are working now.  If you try to access a realm that doesn't exist, there's a quick link to create it.

Update the homepage with a demo interface and topic links are working

New features:
Submit topic
Topic image thumbnail is generated automatically
Subscribe/unsub to realms

I think it would also be cool to let users have avatars.

Great idea, thanks! I put it down for the beta
https://github.com/demcra/demcra/issues/6
Do you like the way github does it? That's what I have in mind

New features:
comment post and voting - basic functionality, but layout needs work
qr code generator - used to deposit cryptocurrency, examples on donate page

You should have the comment fill-in field appear below the comment. Either that or as a "pop-up". Quotes since it would be in the same window. Maybe a transparent overlay above the comments with a "Your comment is here" sort of text or graphic in the location the comment will be posted.
Also a cancel button.

For comments, I plan to put the comment box inline like you mentioned, I should have that done tomorrow.  The way it works now will be the fallback for someone without javascript.
I uploaded the site images so they can be updated here then I'll pull to the live site.

Comment box is now inline and  layout improved a bit.

Is comment editing added yet?

Edit/delete for comments is in now.

Edit/delete for topic summary is in.

By default, topics are now sorted by "popularity", currently defined as:
(vote_score + comment_count) / (1 + seconds_since_posted)

Sorting is in
Calling the prototype done, will add leftover features I wanted as new issues


Change directory structure to upload perf-test results based on job's build number

Description
Currently there is no way to map perf-test build number to the test results uploaded in the s3. This PR changes the path for uploading the test-results from distribution build number to job's build number https://build.ci.opensearch.org/job/perf-test/
Also fixes the tests which was showing null for environment variable $BUILD_NUMBER
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2488 (4b2d5c7) into main (8eb2aeb) will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #2488      +/-   ##
============================================
+ Coverage     94.63%   94.65%   +0.02%
============================================
  Files           213      162      -51
  Lines          4324     4269      -55
  Branches         29       19      -10
============================================
- Hits           4092     4041      -51
- Misses          226      228       +2
+ Partials          6        0       -6

Impacted Files
Coverage Δ

src/system/temporary_directory.py
85.71% <0.00%> (-11.43%)
:arrow_down:

src/system/os.py
93.75% <0.00%> (-6.25%)
:arrow_down:

tests/jenkins/jobs/ScanDockerImage_Jenkinsfile

...ualifier_actions_OpenSearch_Dashboards_Jenkinsfile

...ts/jenkins/jobs/uploadMinSnapshotsToS3_Jenkinsfile

...RunBwcTestScript_OpenSearch_Dashboards_Jenkinsfile

tests/jenkins/jobs/PromoteArtifacts_Jenkinsfile

...ts/jenkins/jobs/BuildUploadManifestSHA_Jenkinsfile

...nkins/jobs/RpmDashboardsDistValidation_Jenkinsfile

tests/jenkins/jobs/BuildDockerImage_Jenkinsfile

... and 54 more

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


[Bug]: xml map operation returns nested xml sequence when invoked with an xml element sequence with more than one element

Description
Consider the following code,
import ballerina/io;

public function main() {
    xml result;

    // this works
    xml<xml:Element> x1 = xml `<item><name>book</name><price>10</price></item>`;
    result = xml:map(xml:elements(x1), v => xml:getChildren(v));
    io:println(result); // <name>book</name><price>10</price>
    io:println(result.length()); // 2
    io:println(result.get(0)); // <name>book</name>
    io:println(result.get(1)); // <price>10</price>

    // nested result
    xml<xml:Element> x2 = xml `<item><name>book</name><price>10</price></item><item><name>pen</name><price>12</price></item>`;
    result = xml:map(xml:elements(x2), v => xml:getChildren(v));
    io:println(result); // <name>book</name><price>10</price><name>pen</name><price>12</price>
    io:println(result.length()); // 2

}

This map operation should result in an flatten xml sequence.
The current output is a nested result when there are more than one element in the sequence. The expected behavior works when there is only one item in the sequence
Steps to Reproduce
No response
Affected Version(s)
2201.9.0
OS, DB, other environment details and versions
No response
Related area
-> Runtime
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

From the sequence value section in the spec,

A sequence value is iterable: the iteration sequence consists of the singleton items of the sequence value in order and the iteration completion value is always nil.

From the iterable sectionin the spec,

For an xml sequence, we iterate over xml singleton values and the type descriptor is equal to xml and the type of the singleton item is T.

From the xml section in the spec,

Note that xml<xml> is the same as xml and that xmlxml:Text is the same as xml:Text.

Since xml<xml> is equal to xml we can conclude that an xml sequence cannot have an xml sequence as the element.
Thus the xml sequence output from the map operation cannot have nested xml sequences and the output of should be a flattened one


Help Remove Search and Fix Number of records per page HOW?

Hi im new to laravel backpack and crud
right now ive tested the "how to use (full example)" method and i want to customized the user interface. i would like to know if its possible to remove the search and records per page then limit the showing entries to maximum of 5 data only with a specific order?

@ajamandre
To limit the entries per page you can change the config default_page_length in config/backpack/crud to the number you want.
In terms of removing the search function, I think this is a feature of datatable and as far I as I see there is currently no way to easily disable this feature.
However, once you have published the view from backpack/crud to your application, you can do whatever you want to with those views.

@yingdongzhang
thank you so much


C++/CLI dot net core 3.1 Utility style projects

I'm not sure if this is the correct project to post this issue but it seemed more likely to be msbuild related than dotnet coreclr.
Anyway I recently ported an existing C++/CLI vcxproj  based solution to build using DotNet Core 3.1 and it mostly worked which is awesome.
However a couple of projects failed with the build error

C++/CLI projects targeting .NET Core must be dynamic libraries.
These project turned out to be

  <PropertyGroup>
    <ConfigurationType>Utility</ConfigurationType>
  </PropertyGroup>

So they were really just bags of files and not building anything. I can understand the C++/CLI support not building all project types but can it allow "utility" style projects? Given that they don't actually build anything I was a bit surprised they weren't allowed.

@wli3 can you take a look at this?

@davidhunter22 sorry we don't support it at this time. cc @olgaark


⚠ Ixtanet SAT has degraded performance

In f9e1884, Ixtanet SAT ($MEDIA_SAT) experienced degraded performance:

HTTP code: 200
Response time: 20641 ms

Resolved: Ixtanet SAT performance has improved in aeb1ba4 after 290 days, 22 hours, 4 minutes.


Text-mask with angular universal (server side rendering)

Angular server side rendering throws an exception: ReferenceError: document is not defined

Please check back once #787 is merged and published. Reopen the issue if the problem persists.


TypeError: Can't convert Operation 'MutableHashTable' to Tensor

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Mac
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): No
TensorFlow version (use command below): 1.8.0
Python version: 3.6.3
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the current behavior
meta graph exported by export_meta_graph cannot be imported by import_meta_graph
Describe the expected behavior
export_meta_graph and import_meta_graph works.
Code to reproduce the issue
import tensorflow as tf
from tensorflow.contrib.lookup.lookup_ops import MutableHashTable
from tensorflow.contrib.lookup.lookup_ops import MutableDenseHashTable

export_dir = 'minimal_saved_model'
#builder = tf.saved_model.builder.SavedModelBuilder(export_dir)

with tf.Session(graph=tf.Graph()) as sess:

    #table = MutableDenseHashTable(key_dtype=tf.int64, \
    # value_dtype=tf.int64, default_value=-1, empty_key=0)
    table = MutableHashTable(key_dtype=tf.int64, value_dtype=tf.int64, default_value=-1)
    aa = tf.get_variable('xxx', shape=(3,))

    meta = tf.train.export_meta_graph()
    saver = tf.train.Saver()

with tf.Session(graph=tf.Graph()) as sess1:
    tf.train.import_meta_graph(meta)

Other info / logs
related to #11888 . It seems that SaveableObject is not recreated correctly.
stack traces:
  File "minimal_case.py", line 22, in <module>
    tf.train.import_meta_graph(meta)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1970, in import_meta_graph
    return Saver()
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1338, in __init__
    self.build()
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1347, in build
    self._build(self._filename, build_save=True, build_restore=True)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1384, in _build
    build_save=build_save, build_restore=build_restore)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 813, in _build_internal
    saveables = self._ValidateAndSliceInputs(names_to_saveables)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 661, in _ValidateAndSliceInputs
    names_to_saveables = BaseSaverBuilder.OpListToDict(names_to_saveables)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 629, in OpListToDict
    var = ops.internal_convert_to_tensor(var, as_ref=True)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1104, in internal_convert_to_tensor
    ret = conversion_func(value, dtype=dtype, name=name, as_ref=as_ref)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 6130, in _operation_conversion_error
    name, as_ref))
TypeError: Can't convert Operation 'MutableHashTable' to Tensor (target dtype=None, name=None, as_ref=True)

saver = tf.train.Saver()
meta = tf.train.export_meta_graph(saver_def=saver.as_saver_def())

I think we could close this. Also we don't support TF 1.x anymore.

Hello, i got the same problem. Did you have a solution?

Hello, i got the same problem. Did you have a solution?

@kkzheng Not yet. But I think what you can do it to replace import_meta_graph call by yourself. I encounter the problem when I try to import and export again in order to manipulate the graph. Finally, I gave it up.

I replicated this problem using compatibility mode in 2.8.

The problem is that we don't have an import test for MutableHashTable but only for StaticHashTable:
https://github.com/tensorflow/tensorflow/blob/480dd2f0f7dbd6da754b714925f7f4623472ad89/tensorflow/python/kernel_tests/data_structures/lookup_ops_test.py#L413-L431

Now it is covered by an expected failing test https://github.com/tensorflow/tensorflow/pull/55200 that we could merge.
We could investigate later how to fix it and collect some hints from the internal team on what kind of PR they want.

This is now covered by an expected failing test.
To contribute this feature with a PR a contributor will need to write the code and pass this test removing the @unittest.expectedFailure decorator.
https://github.com/tensorflow/tensorflow/blob/3cb23b56712d2e85e0a97c58cac771dcf40ca7cb/tensorflow/python/kernel_tests/data_structures/lookup_ops_test.py#L3358-L3381
@mihaimaruseac @theadactyl @yarri-oss  Do you think that we could use a special label other the eventually stat:contributions welcome when we have a ticket covered by an expected failing test like e.g stat:test_covered?
Cause I think It could be really useful for two main goals:

to set a potential preference on contributing a PR cause we have already a specific that it needs to pass.
for the @tensorflow/dev-support to not periodically go  to manually check an often not "minimized" gist on Colab against nightly and new  TF releases to verify that the bug/FR wasn't "silently" solved.

Can we introduce this specific label for test covered contribution?
@MichaelHudgins @rishikasinha-tf @learning-to-play
/cc @mihaimaruseac

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.
Thank you!


Prevent (re-)delivering RabbitMQ messages to Celery workers that're being terminated

This is the same as https://github.com/hypothesis/lms/issues/5304 but for Checkmate.

Closing as won't fix, same as we did for https://github.com/hypothesis/lms/issues/5304


How to use Certificate received from Azure Key-Vault as Client-Certificate under SSL Context

Query/Question
How to use certificate received from Azure Key-Vault as Client-Certificate under SSL Context
Why is this not a Bug or a feature Request?
This is a query on how to use Certificate from Key-Vault as Client-Certificate under SSL Context. Error received from Server is as if no client certificate got actually attached under SSL Context.
When same certificate is used from a resource folder location and used in same code, code works fine.
So, doesn't seems to be a bug or new feature but more on request for guideline on some missing steps.
Setup (please complete the following information if applicable):

OS: Windows 10
IDE: IntelliJ, Eclipse
Library/Libraries: spring-boot-starter-parent: 2.6.7, azure-identity: 1.3.1, azure-security-keyvault-certificates: 4.1.8, spring-boot-maven-plugin

Following code is being used to fetch cert from Key-Vault and use the same as Client Certificate:
   	KeyVaultCertificateWithPolicy certificateWithPolicy = certificateClient.getCertificate(certName);

   	System.out.printf("\nRecevied certificate with name \"%s\", "
   			+ "version %s and secret id %s%n"
   			+ "expiry date: %s%n"
   			+ "Issuer: %s%n",
   	    certificateWithPolicy.getProperties().getName(), certificateWithPolicy.getProperties().getVersion(),
   	    certificateWithPolicy.getSecretId(),
   	    certificateWithPolicy.getProperties().getExpiresOn(),
   	    certificateWithPolicy.getPolicy().getIssuerName()
   	    );

   	byte[] cerContent = certificateWithPolicy.getCer();

   	SSLConnectionSocketFactory socketFactory = null;
   	CloseableHttpClient httpClient = null;
   	try {
   		System.out.println(" Inside TRY Block");
   		CertificateFactory certFactory = CertificateFactory.getInstance("X.509");
   		X509Certificate certificate =
   			(X509Certificate) certFactory.generateCertificate(new ByteArrayInputStream(cerContent));

   		KeyStore azureKeyVaultKeyStore = KeyStore.getInstance("PKCS12");
   		azureKeyVaultKeyStore.load(null, null);
   		azureKeyVaultKeyStore.setCertificateEntry("<name>", certificate);
   		System.out.println(azureKeyVaultKeyStore
   				.isCertificateEntry("<name>"));

   		System.out.println(" Cert Loaded Completed "+ certificate.getType()+"  Content ="+ certificate.getPublicKey() );

   		SSLContext sslContext = SSLContexts.custom()
   				.setKeyStoreType("PKCS12")
   				.loadKeyMaterial(azureKeyVaultKeyStore, null)
   				.build();

   		SSLConnectionSocketFactory sslConFactory =
   				new SSLConnectionSocketFactory(sslContext,allowAll);

   		httpClient = HttpClients.custom().setSSLSocketFactory(sslConFactory).build();

   		HttpComponentsClientHttpRequestFactory requestFactory = new HttpComponentsClientHttpRequestFactory();
   		requestFactory.setHttpClient(httpClient);
   		RestTemplate restTemplate = new RestTemplate(requestFactory);
   		String sslTest = "https://<URI>";
   		String body = "{\"Hello\": 124}";
   		HttpHeaders headers = new HttpHeaders();

   		headers.set("Accept", "application/json");
   		headers.set("Content-Type", "application/json");
   		headers.set("Hello", "124");

   		HttpEntity<String> request = new HttpEntity<String>(body, headers);

   		try {
   		ResponseEntity<String> result = restTemplate.postForEntity(sslTest, request, String.class);
   		System.out.println(result.getStatusCodeValue() + " Body:" + result.getBody());
   		} catch (Exception e) {
   			e.printStackTrace();
   		}

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ Y ] Query Added
[ Y ] Setup information Added

Can someone please guide on this item?

There's a azure-keyvault-jca can help with such case:

https://docs.microsoft.com/azure/developer/java/spring-framework/configure-spring-boot-starter-java-app-with-azure-key-vault-certificates
https://github.com/Azure-Samples/azure-spring-boot-samples/tree/spring-cloud-azure_v4.3.0/keyvault/azure-securtiy-keyvault-jca/run-with-command-line-client-side

Thanks @saragluna .
These links did not have actual fix but did help me guide to the right direction.
Thanks a lot for the help.


Any way of getting the liveId of a console?

Hello, I was wondering if it was possible to get the liveId of a console?
Using a SmartGlassClient or smth

Yep, its a little bit complicated for now tho.
SmartGlass.Device dev = /* get SmartGlass.Device via discover */;
var liveId = dev.Certificate.SubjectDN.GetValueList()[0];


No permissions for plugin

I can't disallow manage editors for users or groups, but it is very important for me

I second this. The absence of the integration of permissions means that it is currently impossible to prevent some backend users (e.g. content editors) access to the WYSIWYG settings.
Please implement permissions.


Adding a method to update stories

@thegrubbsian I took some of your ideas here and added them to code I already had in progress. Please see latest for how to update stories.


feat(multiselect): set some elements as slot to promote customization

This PR exposes selection, max-selection, select-all and option item label content as slot to accommodate custom rendering needs. Also created two slot position in option to enable prepend and appending custom to item list.
Storybook story and vuepress docs for this component has been updated to reflect these changes.

@gravitano might want to check into slot naming. I didn't see any rule about slot naming for this project, so idk if the slot names I set up in this PR are okay or should be changed to a specific pattern.


.prototools file search should recurse up to every parent directory to $HOME/.prototools

What version?
No response
Which command?
No response
What happened?
I have .prototools at $HOME/.prototools yet when going into a ~/code/go/version17/ I get
Error: proto::tool::required
  x This project requires Go 1.17.0, but this  version has not been installed. Install it with proto install go 1.17.0

So two things here.
1, $HOME/.prototools - speciifies go = "1.19"
2. I have go 1.17.13 installed, IMOHO proto should be less strict and be happy that 1.17.13 essentially satisfies go.mod 1.17. Can we just adhere to checking the version range specified?
Any logs?
No response
Operating system?
MacOS
Architecture?
x64

This  also would be more like previous tools like nvm via .nvmrc

@nmccready Can you paste the --log trace or PROTO_LOG=trace?
So we do traverse upwards till we find something, so I'm curious which branch is being hit: https://github.com/moonrepo/proto/blob/master/crates/core/src/version_detector.rs#L45
And also 1.17.13 should satisfy the 1.17 requirement. Something else may be happening here.

Looks to take go.mod over ~/.prototools, it never gets to recursing up further.
$ proto run go --log trace
[DEBUG 2023-09-21 20:50:25] proto  Running proto v0.18.2
[TRACE 20:50:25] starbase::app  Running startup phase with 1 systems
[TRACE 20:50:25] starbase::app  Running analyze phase with 1 systems
[TRACE 20:50:25] starbase::app  Running execute phase with 2 systems
[DEBUG 20:50:25] proto_core::user_config:run:load  Loading config.toml  file="/Users/nem/.proto/config.toml"
[TRACE 20:50:25] starbase_utils::fs:run:load  Opening file  file="/Users/nem/.proto/config.toml"
[TRACE 20:50:25] starbase_utils::fs_lock:run:load  Locking file  file="/Users/nem/.proto/config.toml"
[TRACE 20:50:25] starbase_utils::fs_lock:run:load  Unlocking file  file="/Users/nem/.proto/config.toml"
[DEBUG 20:50:25] proto_core::tool_loader:run  Traversing upwards to find a configured plugin  tool="go"
[TRACE 20:50:25] warpgate::loader:run  Creating plugin loader  cache_dir="/Users/nem/.proto/plugins"
[TRACE 20:50:25] warpgate::loader:run  Loading plugin go  plugin="go" locator="source:https://github.com/moonrepo/go-plugin/releases/latest/download/go_plugin.wasm"
[TRACE 20:50:25] starbase_utils::fs:run  Reading file metadata  file="/Users/nem/.proto/plugins/go-latest-58bf90c564c6b7567d840910e71ad82280c7da85922d52dec948d8d5d60eeaf0.wasm"
[TRACE 20:50:25] warpgate::loader:run  Plugin already downloaded and cached  plugin="go" path="/Users/nem/.proto/plugins/go-latest-58bf90c564c6b7567d840910e71ad82280c7da85922d52dec948d8d5d60eeaf0.wasm"
[DEBUG 20:50:25] proto_core::tool_loader:run  Loading WASM plugin  source="/Users/nem/.proto/plugins/go-latest-58bf90c564c6b7567d840910e71ad82280c7da85922d52dec948d8d5d60eeaf0.wasm"
[DEBUG 20:50:25] proto_core::tool:run  Creating tool go and instantiating plugin
[DEBUG 20:50:25] proto_core::tool_manifest:run:load  Loading manifest.json  file="/Users/nem/.proto/tools/go/manifest.json"
[TRACE 20:50:25] starbase_utils::fs:run:load  Opening file  file="/Users/nem/.proto/tools/go/manifest.json"
[TRACE 20:50:25] starbase_utils::fs_lock:run:load  Locking file  file="/Users/nem/.proto/tools/go/manifest.json"
[TRACE 20:50:25] starbase_utils::fs_lock:run:load  Unlocking file  file="/Users/nem/.proto/tools/go/manifest.json"
[DEBUG 20:50:25] proto_core::tool:run  Created tool go and its WASM runtime
[TRACE 20:50:25] warpgate::plugin:run  Calling plugin function register_tool  plugin="go" input={"id":"go"}
[TRACE 20:50:25] warpgate::plugin:run  Called plugin function register_tool  plugin="go" output={"env_vars":[],"inventory":{"disable_progress_bars":false},"name":"Go","plugin_version":"0.3.0","type_of":"Language"}
[TRACE 20:50:25] proto_core::version_detector:run  Attempting to find local version from config files  tool="go"
[TRACE 20:50:25] proto_core::version_detector:run  Checking directory  tool="go" dir="/Users/nem/code/nmccready/takeout"
[TRACE 20:50:25] warpgate::plugin:run  Calling plugin function detect_version_files  plugin="go" input={}
[TRACE 20:50:25] warpgate::plugin:run  Called plugin function detect_version_files  plugin="go" output={"files":["go.mod","go.work"]}
[TRACE 20:50:25] proto_core::tool:run  Attempting to detect a version from directory  tool="go" dir="/Users/nem/code/nmccready/takeout"
[TRACE 20:50:25] starbase_utils::fs:run  Reading file  file="/Users/nem/code/nmccready/takeout/go.mod"
[TRACE 20:50:25] warpgate::plugin:run  Calling plugin function parse_version_file  plugin="go" input={"content":"module github.com/nmccready/takeout\n\ngo 1.19\n\nrequire (\n\tgithub.com/bogem/id3v2 v1.2.0\n\tgithub.com/dhowden/tag v0.0.0-20220618230019-adf36e896086\n\tgithub.com/spf13/cobra v1.7.0\n\tgithub.com/stretchr/testify v1.8.4\n\tgolang.org/x/oauth2 v0.10.0\n)\n\nrequire (\n\tgithub.com/davecgh/go-spew v1.1.1 // indirect\n\tgithub.com/fatih/color v1.9.0 // indirect\n\tgithub.com/golang/protobuf v1.5.3 // indirect\n\tgithub.com/mattn/go-colorable v0.1.4 // indirect\n\tgithub.com/mattn/go-isatty v0.0.11 // indirect\n\tgithub.com/nmccready/colorjson v0.1.0 // indirect\n\tgithub.com/patrickmn/go-cache v2.1.0+incompatible // indirect\n\tgithub.com/pmezard/go-difflib v1.0.0 // indirect\n\tgolang.org/x/net v0.12.0 // indirect\n\tgolang.org/x/sys v0.10.0 // indirect\n\tgolang.org/x/text v0.11.0 // indirect\n\tgoogle.golang.org/appengine v1.6.7 // indirect\n\tgoogle.golang.org/protobuf v1.31.0 // indirect\n\tgopkg.in/yaml.v3 v3.0.1 // indirect\n)\n\nrequire (\n\tgithub.com/inconshreveable/mousetrap v1.1.0 // indirect\n\tgithub.com/nmccready/go-debug v0.5.0\n\tgithub.com/spf13/pflag v1.0.5 // indirect\n)","file":"go.mod"}
[TRACE 20:50:25] warpgate::plugin:run  Called plugin function parse_version_file  plugin="go" output={"version":"1.19.0"}
[DEBUG 20:50:25] proto_core::tool:run  Detected a version  tool="go" file="/Users/nem/code/nmccready/takeout/go.mod"
[DEBUG 20:50:25] proto_core::version_detector:run  Detected version from tool's ecosystem  tool="go" version="1.19.0"
[DEBUG 20:50:25] proto_core::user_config:run:load  Loading config.toml  file="/Users/nem/.proto/config.toml"
[TRACE 20:50:25] starbase_utils::fs:run:load  Opening file  file="/Users/nem/.proto/config.toml"
[TRACE 20:50:25] starbase_utils::fs_lock:run:load  Locking file  file="/Users/nem/.proto/config.toml"
[TRACE 20:50:25] starbase_utils::fs_lock:run:load  Unlocking file  file="/Users/nem/.proto/config.toml"
[DEBUG 20:50:25] proto_core::tool:run  Resolving a semantic version or alias  tool="go" initial_version="1.19.0"
[DEBUG 20:50:25] proto_core::tool:run  Loading available versions  tool="go"
[TRACE 20:50:25] starbase_utils::fs:run  Reading file metadata  file="/Users/nem/.proto/tools/go/remote-versions.json"
[DEBUG 20:50:25] proto_core::tool:run  Loading from local cache  tool="go" cache="/Users/nem/.proto/tools/go/remote-versions.json"
[TRACE 20:50:25] starbase_utils::fs:run  Opening file  file="/Users/nem/.proto/tools/go/remote-versions.json"
[TRACE 20:50:25] starbase_utils::fs_lock:run  Locking file  file="/Users/nem/.proto/tools/go/remote-versions.json"
[TRACE 20:50:25] starbase_utils::fs_lock:run  Unlocking file  file="/Users/nem/.proto/tools/go/remote-versions.json"
[DEBUG 20:50:25] proto_core::tool:run  Resolved to 1.19.0  tool="go" version="1.19.0"
[DEBUG 20:50:25] proto_core::tool:run  Checking if tool is installed  tool="go" install_dir="/Users/nem/.proto/tools/go/1.19.0"
[DEBUG 20:50:25] proto_core::tool:run  Tool has not been installed  tool="go"
[TRACE 20:50:25] starbase::app  Running shutdown phase with 1 systems
Error: proto::tool::required

  × This project requires Go 1.19.0, but this version has not been installed. Install it with proto install go 1.19.0!

It would be fine if parsing go.mod from  1.19 to go = "1.19" but it does go = "1.19.0"` which is why this is an issue to begin with.

Called plugin function parse_version_file  plugin="go" output={"version":"1.19.0"}

This is coming from the go plugin, I can fix it there.

Ok this should fix it: https://github.com/moonrepo/go-plugin/pull/8
Once that lands, you can delete the plugin in ~/.proto/plugins and try again.

Can ~/.proto/plugins removal be a default in the curl installation script ?

Confirmed, works 🎆

I'm going to have install auto-clean the plugins folder in hte next release.


Is it running autoregressively?

The code seems to be working properly, but I have a question about how it works. I run this:
python benchmarks/benchmark_generation_mamba_simple.py --model-name "state-spaces/mamba-2.8b" --prompt "The climate of Tokyo is" --topp 0.9 --temperature 0.7 --repetition-penalty 1.2
And it produces 100 tokens, which decode to The climate of Tokyo is very hot and humid, with an average temperature in January being around. The city's annual rainfall averages, but can reach as high  in the summer.\n\nHistory \n was founded on April 1st 1871 by a group led mainly from Kōshū (now part of Shizuoka Prefecture) who were dissatisfied that their local government had been taken over after it became independent following its secession during Meiji Restoration; they wanted to create another prefectural capital for.
I modified benchmark_generation_mamba_simple.py to just call fn() once instead of repeats+1 times.
I was curious how many times it calls the forward() pass for one Mamba layer. So, I made this small code change:
class Mamba(nn.Module):
  ...
def forward(self, hidden_states, inference_params=None):
  ...
  if self.layer_idx == 0: print("Mamba: calling forward on layer 0")

I thought I would see Mamba: calling forward on layer 0 print 100 times, since it  produces 100 new tokens. But, what surprised me is that it only prints Mamba: calling forward on layer 0 4 times.
What I am wondering is, shouldn't it call the layer once per token? Or... how does the autoregressive inference work for this particular model?

We cache the computation graph so not all the prints will show up. You can set cg=False in generate to disable this caching, and all the prints should show up.

Thank you for the fast and detailed response! That worked! With cg=False, I see 100 model executions to generate 100 tokens.


Fixed issue with System.Text.Json

Starting with DotNetCore 3, the new System.Text.Json namespace is used by default to deserialize objects from requests in controllers instead of Json.Net. For the most part this doesn't matter but it does when it comes to how lists are deserialized into objects. This in turn affects the JsonNetFilterRule object's "Value" property when passing lists for operators like "In".
Here is how Json.Net will parse string lists (from the tests for InClause()):

And here is how System.Text.Json does the same:

When parsed by this library, the two values are interpreted differently, the System.Text.Json version ends up with an additional quote in the resulting query (S.T.Json version first, Json.Net seccond):

This in turn means that the query is not applied as expected and no results are returned!
I have traced this particular issue with string lists to the "GetConstants" function and believe it will be very easy to resolve by simply adding an additional trim which will remove surrounding quotes.
In order to test this issue I have added an additional Nunit testing project to the solution which is built on DotNetCore 3.1 (so that it can use the System.Text.Json namespace). I made some changes to the existing testing project so that I could reference and re-use some methods in the new project.
In the new testing project I re-created the "InClause" test and passed each filter through the System.Text.Json serializer to model the process of ASP.NetCore 3+ deserializing  an incoming request. Adding the additional trim operation fixes the issues and all existing tests still pass.
Note that I did remove the "nullableWrappedStatValueFilter" portion of the test. I'm not certain about the need for the "Wrapper" but it causes a nested "{"value": foo}" when using System.Text.Json. From what I can tell, the need for the Wrapper is based on Json.Net and may not be necessary for System.Text.Json. I would like your input on this, it is certainly possible to add extra logic to unwrap the extra object notation but I didn't want to do too much in one PR.
Note that there is some risk in this change in that if your query included quotes at the beginning or end of the value those will be stripped since Trim() removes all occurrences from the beginning and end. If you feel this is unacceptable, we can add some extra logic to check for a single quote at the beginning and end of the value and remove only those quotes using substring or something.

@gomjabar6 thanks for the PR here as well.  I'll take a little time to review, the JsonNetFilterRule is an implementation of IFilterRule that was designed explicitly to play nice with JSON.Net serialization.  There's the QueryBuilderFilterRule implementation of IFilterRule, which if memory serves, worked with the then current System.Text.Json when it was .Net 4.5.  In short, realizing we shouldn't try and force a dependency on one serializer vs another, the idea at least, was to allow for serializer specific versions of the IFilterRule implementation so it can play nice everywhere.  Will investigate a little and report back.

@gomjabar6 thanks for the PR here as well.  I'll take a little time to review, the JsonNetFilterRule is an implementation of IFilterRule that was designed explicitly to play nice with JSON.Net serialization.  There's the QueryBuilderFilterRule implementation of IFilterRule, which if memory serves, worked with the then current System.Text.Json when it was .Net 4.5.  In short, realizing we shouldn't try and force a dependency on one serializer vs another, the idea at least, was to allow for serializer specific versions of the IFilterRule implementation so it can play nice everywhere.  Will investigate a little and report back.

@tghamm, thanks for the info about the history of JsonNetFilterRule and QueryBuilderFilterRule. We tried to use QueryBuilderFilterRule first but apperently the new System.Text.Json behaves similarly to Json.Net in that it won't deserialize non strings into a string[] which is how QueryBuilderFilterRule works:

I can see what you mean about creating serializer specific versions for IFilterRule... however I can't come up with anything different to do in a DotNetCore3+ version versus what has been done in the JsonNetFilterRule already (making the Value "Object" typed).
It is possible that there are some configuration options in DotNetCore3+ System.Text.Json which would make it behave more like Json.Net with respect to Object deserialization. If so perhaps that could be a path forward without needing to change this library. I'll do some more research tomorrow.

@tghamm, thanks for the info about the history of JsonNetFilterRule and QueryBuilderFilterRule. We tried to use QueryBuilderFilterRule first but apperently the new System.Text.Json behaves similarly to Json.Net in that it won't deserialize non strings into a string[] which is how QueryBuilderFilterRule works:

I can see what you mean about creating serializer specific versions for IFilterRule... however I can't come up with anything different to do in a DotNetCore3+ version versus what has been done in the JsonNetFilterRule already (making the Value "Object" typed).
It is possible that there are some configuration options in DotNetCore3+ System.Text.Json which would make it behave more like Json.Net with respect to Object deserialization. If so perhaps that could be a path forward without needing to change this library. I'll do some more research tomorrow.

@tghamm, found Microsoft's take on this: https://docs.microsoft.com/en-us/dotnet/standard/serialization/system-text-json-migrate-from-newtonsoft-how-to?pivots=dotnet-5-0#deserialization-of-object-properties
Perhaps we could do a custom object which will unpack the "json element" automatically.

@tghamm, found Microsoft's take on this: https://docs.microsoft.com/en-us/dotnet/standard/serialization/system-text-json-migrate-from-newtonsoft-how-to?pivots=dotnet-5-0#deserialization-of-object-properties
Perhaps we could do a custom object which will unpack the "json element" automatically.

@tghamm, Okay I made a pass at a SystemTextJsonFilterRule. I had to get fancy with the multitargeting since net45 does not have access to System.Text.Json (which is necessary for this class) but I think it all works out great. Essentially if you are on a net45 project, the new class will not exist but on netstandard2.0 it will become available to use.
My solution was to use a getter on the Value to check if the object is an JsonElement enumerable. If so it will use the given rule.Type to convert the JsonElement into the appropriately typed Array and thus setup for the QueryBuilder to parse. I have implemented all of the types listed in the Type comments, to do so I have broken out your existing Type switch statement from QueryBuiler into a static function.
I have copied all of the existing SystemNetJson tests into the new DotNetCore3.1 testing project. Using the new SystemTextJosnFilterRule all tests pass, including all existing tests. I was even able to undo the Trim('"') change to QueryBuilder.
Some things to consider:

I'm not an expert on multitargeting and I'm not sure if the way I did it means the new features will be available on DotNet 5 (which they will work on). It would be good if someone who has more experience in multitargeting can review the package.
There may be some changes needed to support cultures. I had to do date and number parsing and I know sometimes this can be effected. Again, it would be good to have an expert in cultures review this as well.

@tghamm, Okay I made a pass at a SystemTextJsonFilterRule. I had to get fancy with the multitargeting since net45 does not have access to System.Text.Json (which is necessary for this class) but I think it all works out great. Essentially if you are on a net45 project, the new class will not exist but on netstandard2.0 it will become available to use.
My solution was to use a getter on the Value to check if the object is an JsonElement enumerable. If so it will use the given rule.Type to convert the JsonElement into the appropriately typed Array and thus setup for the QueryBuilder to parse. I have implemented all of the types listed in the Type comments, to do so I have broken out your existing Type switch statement from QueryBuiler into a static function.
I have copied all of the existing SystemNetJson tests into the new DotNetCore3.1 testing project. Using the new SystemTextJosnFilterRule all tests pass, including all existing tests. I was even able to undo the Trim('"') change to QueryBuilder.
Some things to consider:

I'm not an expert on multitargeting and I'm not sure if the way I did it means the new features will be available on DotNet 5 (which they will work on). It would be good if someone who has more experience in multitargeting can review the package.
There may be some changes needed to support cultures. I had to do date and number parsing and I know sometimes this can be effected. Again, it would be good to have an expert in cultures review this as well.

I've merged this in, but added the SystemTextJsonFilterRule as a class in a helper NuGet package called Castle.DynamicLinqQueryBuilder.SystemTextJson.  It added a hard dependency on System.Text.Json which I'd prefer not to have, so I just parceled it out to a second package and updated the documentation to let folks know.  Appreciate the contribution!


Permitting Empty Responses for Server Resource Discovery

Custom metric servers often return an empty response to metric queries as a means to conserve resources. This update logs a warning when such responses are received and then proceeds without further action, aligning with the upstream workaround.
Currently, it is not feasible to develop a valid end-to-end test because the cluster version impacts the reproducibility of the issue, which does not occur on clusters version 1.28 and above. Therefore, only unit tests will be added.

/retest-required

/retest-required

/retest-required

Currently, it is not feasible to develop a valid end-to-end test because the cluster version impacts the reproducibility of the issue, which does not occur on clusters version 1.28 and above. Therefore, only unit tests will be added.

Can we write a test that only runs on certain k8s versions where the issue is reproducible? Conversely, do we need this if the issue isn't reproducible on any tested/supported k8s versions?

Can we write a test that only runs on certain k8s versions where the issue is reproducible? Conversely, do we need this if the issue isn't reproducible on any tested/supported k8s versions?

Just tried again this morning. Master version that can recreate the error (1.27.11) is no longer available, node version still exists, we would specify this cluster config but it might not worth that much. I'm fine of aborting the change since Config Sync has updated k8s library and the k8s 1.28+ also has aggregated discovery, meaning this error is likely no longer an issue.

https://github.com/kubernetes/kubernetes/pull/115978 Should be fixing the issue from memory cached discovery client, as Config Sync has upgraded to over K8s 1.28 the discovery client should no longer receive empty response error for custom metric servers which might not have implemented the resource list.
For cx who are seeing the issue, upgrade Config Sync to over 1.18.3 and upgrade cluster to over 1.28 should fix the issue.


Did not rename ObjC's Swift-code-import-header

In ObjC files that import Swift code, is it possible to have it change:
#import <$(OLD_NAME)-Swift.h> to
#import <$(NEW_NAME)-Swift.h> ?

The script literally goes through (almost) all the files in a directory and replaces all instances of "OLD_NAME" to "NEW_NAME" within content of each file, so the simple answer is: yes, it's possible.

Thank you. For me, that is the only thing it misses. Does it work on those import statements for you?


Update version in spec

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1641

Relevant Lines:
2099

💛  - Coveralls

Just a spec version adjustment, so merged directly.


fix(deps): update terraform terraform-ibm-modules/kms-all-inclusive/ibm to v4.15.12

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/kms-all-inclusive/ibm (source)
module
patch
4.15.11 -> 4.15.12

Release Notes

terraform-ibm-modules/terraform-ibm-kms-all-inclusive (terraform-ibm-modules/kms-all-inclusive/ibm)
v4.15.12
Compare Source
Bug Fixes

deps: update terraform ibm to latest for the deployable architecture solution (#546) (23941aa)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 1.11.12 :tada:
The release is available on:

GitHub release
v1.11.12

Your semantic-release bot :package::rocket:


Allow the pageBuilder to provide the amount of total items

This should be possible so the loading widget isn't shown at the end of the list.

At the moment, the following workaround is an option:
int _totalFeedbackItems;

...

Pagination<Feedback>(
  pageBuilder: (currentSize) async {
    if (currentSize == _totalItems) return const [];
    return await getData(currentSize);
  },
  itemBuilder: ...
);


Error received when trying to navigate to any /Admin/* pages

I am able to log in as<EMAIL_ADDRESS>and<EMAIL_ADDRESS>However, an error is displayed when I try to navigate to any page under /Admin.

I do not get the error when logging in as<EMAIL_ADDRESS>Version: <IP_ADDRESS> ddb2198

Talked with @mheggeseth and I am also not sure if these are exceptions and they are getting swallowed by the code and not logged out anywhere for us to view

@tonysurma This can be closed. Superseded by the more specific #1066.


Flutter Duplicate GoogleService-Info

[✓] Flutter (Channel stable, v1.17.5, on Mac OS X 10.15.4 19E287, locale en-CA)
• Flutter version 1.17.5 at /Users/ivanchau/Documents/flutter
• Framework revision 8af6b2f038 (11 days ago), 2020-06-30 12:53:55 -0700
• Engine revision ee76268252
• Dart version 2.8.4
[✓] Xcode - develop for iOS and macOS (Xcode 11.4.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.4.1, Build version 11E503a
• CocoaPods version 1.8.4
[REQUIRED] Step 2: Describe the problem
I posted this question on flutter's github page but I was asked to direct the question to the firebase team instead. I have a script in the Build Phases that move the GoogleService-Info file to the correct directory depending on the flavor or environment I have chosen.
environment="default"

# Regex to extract the scheme name from the Build Configuration
# We have named our Build Configurations as Debug-dev, Debug-prod etc.
# Here, dev and prod are the scheme names. This kind of naming is required by Flutter for flavors to work.
# We are using the $CONFIGURATION variable available in the XCode build environment to extract
# the environment (or flavor)
# For eg.
# If CONFIGURATION="Debug-prod", then environment will get set to "prod".
if [[ $CONFIGURATION =~ -([^-]*)$ ]]; then
environment=${BASH_REMATCH[1]}
fi

echo $environment

# Name and path of the resource we're copying
GOOGLESERVICE_INFO_PLIST=GoogleService-Info.plist
GOOGLESERVICE_INFO_FILE=${PROJECT_DIR}/Runner/Firebase/${environment}/${GOOGLESERVICE_INFO_PLIST}

# Make sure GoogleService-Info.plist exists
echo "Looking for ${GOOGLESERVICE_INFO_PLIST} in ${GOOGLESERVICE_INFO_FILE}"
if [ ! -f $GOOGLESERVICE_INFO_FILE ]
then
echo "No GoogleService-Info.plist found. Please ensure it's in the proper directory."
exit 1
fi

# Get a reference to the destination location for the GoogleService-Info.plist
# This is the default location where Firebase init code expects to find GoogleServices-Info.plist file
PLIST_DESTINATION=${BUILT_PRODUCTS_DIR}/${PRODUCT_NAME}.app
echo "Will copy ${GOOGLESERVICE_INFO_PLIST} to final destination: ${PLIST_DESTINATION}"

# Copy over the prod GoogleService-Info.plist for Release builds
cp "${GOOGLESERVICE_INFO_FILE}" "${PLIST_DESTINATION}"

Running Xcode build...
Xcode build done.                                           17.9s
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    error: Multiple commands produce '/Users/ivanchau/AndroidStudioProjects/fella/build/ios/Debug-development-iphonesimulator/Runner.app/GoogleService-Info.plist':
    1) Target 'Runner' (project 'Runner') has copy command from '/Users/ivanchau/AndroidStudioProjects/fella/ios/Runner/Firebase/development/GoogleService-Info.plist' to '/Users/ivanchau/AndroidStudioProjects/fella/build/ios/Debug-development-iphonesimulator/Runner.app/GoogleService-Info.plist'
    2) Target 'Runner' (project 'Runner') has copy command from '/Users/ivanchau/AndroidStudioProjects/fella/ios/Runner/Firebase/production/GoogleService-Info.plist' to '/Users/ivanchau/AndroidStudioProjects/fella/build/ios/Debug-development-iphonesimulator/Runner.app/GoogleService-Info.plist'

    note: Using new build system
    note: Building targets in parallel
    note: Planning build
    note: Constructing build description
    warning: duplicate output file '/Users/ivanchau/AndroidStudioProjects/fella/build/ios/Debug-development-iphonesimulator/Runner.app/GoogleService-Info.plist' on task: CopyPlistFile /Users/ivanchau/AndroidStudioProjects/fella/build/ios/Debug-development-iphonesimulator/Runner.app/GoogleService-Info.plist /Users/ivanchau/AndroidStudioProjects/fella/ios/Runner/Firebase/production/GoogleService-Info.plist (in target 'Runner' from project 'Runner')
    warning: Mapping architecture armv7 to i386. Ensure that this target's Architectures and Valid Architectures build settings are configured correctly for the iOS Simulator platform. (in target 'image_picker' from project 'Pods')
    warning: Mapping architecture arm64 to x86_64. Ensure that this target's Architectures and Valid Architectures build settings are configured correctly for the iOS Simulator platform. (in target 'image_picker' from project 'Pods')

Could not build the application for the simulator.
Error launching application on iPhone 11 Pro.

Steps to reproduce:
flutter run --flavor development
I have setup multiple schemes following the instructions in this link: https://medium.com/@aliakhtar_16369/managing-different-environments-using-xcode-build-schemes-and-configurations-af7c43f5be19
Just to be clear, everything was working fine in flutter v1.12 but I just recently upgraded to v1.17 and I'm getting this error when run on iOS. Can someone please let me know what's the best solution to solve this problem? Thanks

@ivanchaukn Did you also change the Xcode version? Does it make a difference to use the same Xcode that was used before the flutter upgrade?

@paulb777  No I did not upgrade my xcode version during my upgrade to Flutter v1.17. I was using Xcode 11.4.1 even with Flutter v1.12

It looks like you've accidentally included both
${PROJECT_DIR}/Runner/Firebase/production/${GOOGLESERVICE_INFO_PLIST}
${PROJECT_DIR}/Runner/Firebase/development/${GOOGLESERVICE_INFO_PLIST}

into your build target. If you open Xcode and search for "GoogleService-Info.plist" in the left navigator filter, do you find two results?

This is intentional. The script from build phase should copy the file from GOOGLESERVICE_INFO_FILE to PLIST_DESTINATION. Please advise if that's not the proper way of handling multiple environments.

You can include them in your Xcode project navigator, but it's important that they are not included in the target membership in Xcode. If they are, Xcode will attempt to copy them into your app bundle. Select both of these files and edit their target membership in the panel in the bottom right corner of Xcode.
Since your script copies the correct plist file directly into the app bundle, you should remove both of these plists from the target.

Thanks! It fixed the first issue but I'm getting another error now. I know this is not a firebase related issue but if you can advise me on how to fix the issue, that will be awesome!
error: Cycle inside Runner; building could produce unreliable results. This usually can be resolved by moving the target's Headers build phase before Compile Sources.
    Cycle details:
    → Target 'Runner': LinkStoryboards
    ○ Target 'Runner' has compile command with input '/Users/ivanchau/AndroidStudioProjects/fella/ios/Runner/Base.lproj/LaunchScreen.storyboard'
    ○ Target 'Runner': Ditto /Users/ivanchau/AndroidStudioProjects/fella/build/ios/Debug-development-iphonesimulator/Runner.swiftmodule/Project/x86_64-apple-ios-simulator.swiftsourceinfo /Users/ivanchau/Library/Developer/Xcode/DerivedData/Runner-gsqfvskdiydqcdftydqhgqbekwhx/Build/Intermediates.noindex/Runner.build/Debug-development-iphonesimulator/Runner.build/Objects-normal/x86_64/Runner.swiftsourceinfo
    ○ Target 'Runner' has compile command for Swift source files
    ○ That command depends on command in Target 'Runner': script phase “Flutter”
    ○ Target 'Runner' has process command with output '/Users/ivanchau/AndroidStudioProjects/fella/build/ios/Debug-development-iphonesimulator/Runner.app/Info.plist'
    ○ Target 'Runner' has compile command with input '/Users/ivanchau/AndroidStudioProjects/fella/ios/Runner/Base.lproj/LaunchScreen.storyboard'
    note: Using new build system
    note: Building targets in parallel
    note: Planning build
    note: Constructing build description
    warning: Mapping architecture armv7 to i386. Ensure that this target's Architectures and Valid Architectures build settings are configured correctly for the iOS Simulator platform. (in target 'image_picker' from project 'Pods')
    warning: Mapping architecture arm64 to x86_64. Ensure that this target's Architectures and Valid Architectures build settings are configured correctly for the iOS Simulator platform. (in target 'image_picker' from project 'Pods')

Unfortunately this is outside my area of expertise. You may find some more help asking on Flutter forums or on StackOverflow with the flutter tag. If you're able to isolate this build error to just Xcode, I can also probably help you further.
I'll close this issue since it's not a Firebase issue, but feel free to continue commenting and I'll help you debug if I'm able.


CardHeader is missing action property

[x] I have searched the issues of this repository and believe that this is not a duplicate.

https://material-ui-next.com/api/card-header/
CardHeader allows to put in the title, subheader, and avatar but no way to add action button.
Yes, i can use workarounds, create some css etc but i believe it should be solved elegantly with action property.

@oliviertassinari can i take a shot at implementing this?

@lukePeavey Great!

@mbrookes ok, i'll give it try!

@mbrookes
@oliviertassinari
I finished an initial draft of the changes.  I just pulled the latest changes from v1-beta so my branch would be up to date. A whole bunch of tests are failing now, but im getting the same result on the latest version of v1-beta. Should i go ahead and submit the PR?

@lukePeavey Feel free to submit your PR so we can take a look and give feedback. If the test failures are coming from the v1-beta branch, you'll be able to rebase once that's resolved, before we merge.

@mbrookes Submitted an initial PR.


Support ReplicasPerMaster property for Redis Cache creation

Added preview param --replicas-per-master to az redis create command and updated azure-mgmt-redis from 6.0.0 to 7.0.0rc1

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

@Juliehzl  please help with the PR.

Minor changes. If all test can pass, I will approve.

Minor changes. If all test can pass, I will approve.

@Juliehzl  All checks have passed now.


chore(master): release 2.2.2

:robot: I have created a release beep boop
2.2.2 (2023-09-22)
Miscellaneous

deps: update actions/checkout action to v4 (fcc445b)
deps: update actions/checkout digest to 8ade135 (a8f8298)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/docker-php-base/releases/tag/v2.2.2 :sunflower:


Inform user if value was changed automatically

Description
The width field (and maybe some others) may adjust the entered value if it's detected as a number, e.g. .4 is changed to 0.4, 08 is changed to 8.
When the value starts with a digit, it is automatically truncated to a number, e.g. 3a is changed to 3.
I believe this automatic truncation may lead to wrong (2 instead of 2,8) or even invalid (0 instead of 0,5) data.
I think it is important to inform the user that the actual stored value is not what was entered, as this is rather unexpected behavior, especially because similar numeric fields like maxwidth, maxweight or maxheight don't do this.
Related issue: #3615 would help in a common case of a wrong decimal separator leading to truncating that value.
Screenshots
No response

When the value starts with a digit, it is automatically truncated to a number, e.g. 3a is changed to 3.

The Width field currently expects a numeric value, so the “a” would be unexpected anyways. ideditor/schema-builder#15 would change this field to a new measurement type, but ideally iD would present a dropdown menu for selecting the unit instead of making you type it out manually.

The Width field currently expects a numeric value

Then simply complain to the user when a non-numeric value is entered. This would also stop users from entering purely non-numeric values (which are not changed at all).

Fixing those problems in the first place would be better than presenting a warning each time it happens.

I would be, but I expect the user will not try to enter 3a over and over if there is a warning every time.
Plus, the problem has been known for ~6 years now and was not fixed, leading to many invalid (0 instead of 0,5) or hard to detect incorrect (2 instead of 2.5) values. So at least the user should be warned if the input is considered invalid until the problem is fixed.

Then simply complain to the user when a non-numeric value is entered. This would also stop users from entering purely non-numeric values (which are not changed at all).

There is some minimal value validation for some fields such as Email. The validation you’re proposing would be implemented similarly. However, it should only be applied to fields whose values must be numeric, not fields whose values are usually numeric but can sometimes be non-numeric, for example Direction.

Plus, the problem has been known for ~6 years now and was not fixed

#8769 is waiting to get merged, so fixing the root cause is even easier than implementing the proposed workaround.


Add LIGHTNING_BENCH_TRACE environment variable.

When set, this will dump the trace for the given benchmark[s]. This was a feature request from a colleague debugging why a trace was slower than expected.
We'd talked about this, and there were some unresolved questions/thoughts:

is printing the trace to a terminal actually the thing we want? A trace alone doesn't let the network be replayed, unfortunately
should we add some sugar to an approach like the above? Like what if the actual pytest benchmark can be imported and asked for its function and inputs, so practitioners don't have to go the benchmark itself?
should we add a notebook about debugging benchmarks to the dev_tutorials directory?

To elaborate on the second option, should we do support something like:
from thunder.benchmarks.targets import foo_bench

fn = foo_bench.fn
args, kwargs = foo_bench.make_inputs()

fn(*args, **kwargs)

thunder.last_traces(fn)

And maybe even more sugar, like
print(foo_bench.source_location)

and that prints a clickable link to the source for the benchmark, or the script the benchmark pulls from, so people can see and modify the code?

Is there a reason to not call it THUNDER_BENCH_TRACE btw?

To elaborate on the second option, should we do support something like:

from thunder.benchmarks.targets import foo_bench

fn = foo_bench.fn
args, kwargs = foo_bench.make_inputs()

fn(*args, **kwargs)

thunder.last_traces(fn)

I think we should this and we do, he's real runnable code:
from thunder.benchmarks import LitGPTSDPABenchmark
import thunder

foo_bench = LitGPTSDPABenchmark("Llama-3-8B", batchdims=(2,))
fn = thunder.jit(foo_bench.fn())
args, kwargs = foo_bench.make_batch()

fn(*args, **kwargs)

thunder.last_traces(fn)

And maybe even more sugar, like
print(foo_bench.source_location)
and that prints a clickable link to the source for the benchmark, or the script the benchmark pulls from, so people can see and modify the code?

It's certainly a useful feature for some people, but there are specialized tools just for searching the source location. I think we can prevent the growth of complexity in Thunder simply by suggesting using these tools.
Python has a built-in library for inspecting the source code of live objects (https://docs.python.org/3/library/inspect.html#retrieving-source-code). Many IDEs have a search feature to quickly find the source location of any object definition within a repo (e.g. https://code.visualstudio.com/docs/editor/codebasics?ref=arctype.com#_search-across-files).

I encourage discussion to continue here, but as per Luca's guidance I am going to close as I do not intend to make time for a 2nd spin in the next ~week.


Set cache option globally for the render method

Is your feature request related to a problem? Please describe.
It could be very long to turn on cache by setting expire time on each wp template.
I'm looking for a way to set cache expire time on the render method globally
Describe the solution you’d like
Set render option in function.php like
Timber::render_cache( $expire, $cache_mode)
Describe alternatives you’ve considered
Or use a wp filter hook
Thank you

Until this lands as a feature, what you can do is wrap Timber::render into a function and use that instead. For example:
<?php
function my_render($templates, $context) {
  return Timber::render($templates, $context, ...cache options...);
}

And do a global replace of Timber::render( with my_render(.

@danburzo nice simple trick.
Thank you.

Closing this as there is a solution provided.


Models imported to s&box don't work

they broke it

Workarounds:

Use an older version of s&box and grab the compiled files.

Use Alyx or Dota, import like usual, for each model convert old to modeldoc, grab the new content files and manually upgrade the modeldoc format so that they're readable.

I tested and it is now working. The editor can also be enabled by running s1import a second time (after importing and doing a full compile) with the temporary fix.


Ability to send additional runtime data

Similar to the gem: https://docs.rollbar.com/docs/ruby#section-including-additional-runtime-data

Implemented in https://github.com/rollbar/rollbar-php/pull/397


[DO NOT MERGE][service-bus] Service bus 1.1.4 patch

Reducing send timeout to 10s. PR created to generate build artifact.

/azp run js - servicebus - ci


Update xmap guide - add jit/shard_map note

Adds a note to the xmap guide:

UPDATE: The recommended ways to do multi-device programming in JAX are using: 1) jit (automatic partitioning of computation and jax.Array sharding); and/or 2) shard_map (manual data sharding). Learn more in Why don’t pmap or xmap already solve this? in the shard_map JEP document.

@jakevdp

Content looks good, but there are documentation build warnings:
/home/docs/checkouts/readthedocs.org/user_builds/jax/checkouts/18022/docs/notebooks/xmap_tutorial.ipynb:10004: ERROR: Document or section may not begin with a transition. I think this comes from starting the section with an hline.

@jakevdp

Oops, looks like the notebook and the md got out of sync. The easiest way to fix that is to run pre-commit run jupytext


Python 3 / Django 2.1.x Support

Is there any timeline when this library will support Python 3 / Django 2.1.x?

I second this.


added ooki

ooki tokens eth and polygon
https://ooki.com

Hi guys,
What's wrong here?
the tests are failed but no details

X 'Check that logos are not too large':  4 errors
X   'Logo too large: /home/runner/work/assets/assets/blockchains/ethereum/assets/0x4A64515E5E1d1073e83f30cB97BEd20400b66E10/logo.png'
X   'Logo too large: /home/runner/work/assets/assets/blockchains/ethereum/assets/0x85D7bdfc9c3426b33A684241eEEE70385Bc42820/logo.png'
X   'Logo too large: /home/runner/work/assets/assets/blockchains/ethereum/assets/0xB5e09e6Bf6a5E96934B3fd99a40F7EDACa1173Ed/logo.png'
X   'Logo too large: /home/runner/work/assets/assets/blockchains/ethereum/assets/0xad3E3Fc59dff318BecEaAb7D00EB4F68b1EcF195/logo.png'

these are not my files, why it failed?

Added