13th Age Glorantha Sheet - Critical damage roll button macro request

Hi there - just kicking off a new 13th Age Campaign. Thanks for all the work on the sheet.
I had a question...
The sheet tracks 'Uses' of a power, both current and max. i.e. I can use this power 6 times in a day, and I've currently used in twice.
When I roll the power I calculate and display the Hit Roll and Damage. Is there a way to also increment the current count field on the character sheet when I 'roll' the power?
Inquiring over on the Roll20 Discord, the suggestion was:
'If the game was created by a pro subscriber, they could install the ammo or chatsetattr scripts to be able to do something like that.  Otherwise, macros can read sheet attributes, but can't alter them.  That is something that generally requires the api.'

No there is no way to do this that I know of, sorry. Not without more open javascript access (which is deliberately very limited or security reasons) or full API access (which is a feature that only pro level subscribers get and I have no idea how to use).

Hello @Seraaron
As this issue's title was very generic, I'll add a comment/question here.
Does the sheet calculate critical damage correctly? To me it seems it isn't, or I am not understanding something correctly.
Character is level 1, his weapon does d6 damage and his STR bonus is 2. He rolls a crit (20), the damage shows 5, calculated like this:
(1*1)d6+2 = (3)+2
Shouldn't the damage be double, i.e. 10?
Thanks!

Good question! I haven't looked at this sheet or played 13th Age in over a year, but I'm currently making a new sheet for another game, so I may be able to add a fix in the not too distant future.
But with a cursory glance (and remembering how I designed this sheet), I think it doesn't automatically calculate crits on purpose. You can go to the settings tab and see how the macros are calculated. Pressing the 'melee' attack button basically just calls @{pma} and @{pmd} in sequence. IRRC there's no easy way to reference a previous roll even from within the same macro. Like, there's no conditional logic in macros I don't think... So I can' write something that says if @{core}==20 then @{pmd}*2
If you know of any other sheet that does this then let me know and I'll have a look at it to see if this is a solved problem and I'm just trying to reinvent the wheel.

Or if there is a way to do it then it's possible that I left it uncalculated on purpose because I seem to remember there being multiple different optional rules in 13A about what to do with crits. So it may be that I figured each group would decide for themselves if crits double or whatever. You can still quite clearly see when a crit should happen from the blue highlighting, at least, and it's not that hard to double a number in your head.

@Seraaron Thanks for the quick answer. I haven't read all the 13th Age Core classes in much detail, but from 13th Age Glorantha I cannot remember there being other effects from crits than doubling the damage. I could be wrong though. What there is though, is that the crit range can differ from character to character, so some characters would crit also on a 19 for example. So in that sense it might be good to never double the damage, and just leave it to the players to figure it out if it is a crit or not.
But now that you mentioned other sheets, I checked how D&D 5E by Roll20 sheet handles crits. It does it so that when you first roll the skill, there is a link for the weapon under the roll and when you click on that, it knows if the damage dice are rolled twice or not (5E damage is little different from 13th Age). Checking the links that the sheet creates under the skill rolls, when on that sheet you roll a crit, there is damage_crit at the end of the link, and damage_dmg with a normal roll, so I am guessing that's how you would need to do it if you would want to.
In the image below 4 is the normal damage is and 6 is from the die allowed by the crit.

But anyway, yes, easy enough to double the damage in your head when needed, agreed!

If I recall correctly the D&D sheet uses some pretty complex API scripts that I have no idea how to reproduce (or if it's even possible for a non-roll20 staff member to reproduce, since the D&D sheet has the official WotC seal of approval). I think the closest I can do is called 'peuso-referencing' which is what you already see, where the macro shows you how your roll is composed without needing to hover over the result with your mouse. But like I said, I'll have a closer look in the not-too-distant future and let you know :)

@Seraaron Yeah, tbh I don't think it would be worth the trouble anyway, probably would be a lot of work and so easy to double the number by yourself. Thanks for the great sheet, we're currently playing the Six Seasons in Sartar campaign with 13th Age rules.

So it looks like there might be a way to do this without API scripting after all, but my current project is more demanding than I thought it would (making a whole new sheet for another game from scratch), be so it might be a little while yet before I come back around to adding this to the glorantha sheet.

@Seraaron No worries, good luck with the new project! What game is it for, if I may ask (just curious) ?

Okay so I finished the ICON sheet and looked at this properly, finally. I seems like my own smartassery is coming back to bite me now. Basically I designed the 13G sheet very early in my web-dev learning experience, and I made a lot of (in hindsight) very weird decisions to work around certain problems and to calculate things in a unique way, because I didn't know any better, while also trying to leave as much of it exposed to be edited and fiddled with in the settings menu. In effect, what this means is that while the sheet is very flexible for the end user, it makes it very difficult for me (or anyone else for that matter) to come back to and edit or add new stuff.
For me to add a chat menu to melee rolls for crits (or my other idea was to conditionally show crit damage if you roll above a certain amount) and still retain all the currently exposed code in the settings menu I think I'd need to rewrite roll macros from scratch... And that would potentially impose some 'breaking changes' that would affect all sheet users at this time. Ideally, the whole sheet would need to be updated from legacy mode and then it could take advantage of some new roll20 features, like custom roll parsers. But that would be a herculean task. It would probably be easier to make a whole new sheet honestly.
My pro subscription that gives me access to the shet development envrionment expires on Wednesday, and I have other projects to work on in Feb. So I think I'm gunna close the issue say that people can just double the numbers in their heads.

@Seraaron I understand totally, being a developer myself and still at times having to work with some legacy stuff that I coded decades ago :) Also, our gaming group in the end decided to change the system to HeroQuest Glorantha so for us this wasn't an issue anymore anyway (our decision to change the system had nothing to do with Roll20 or the character sheets). Your 13G character sheet is great and would recommend it to others. Thanks and all the best going forward!

Cheers!


Support for physical vs logical pixels

On macOS, the buffer is applied using logical pixels rather than physical pixels so a 100x100 buffer will fill a 100x100 (logical pixels) window even if the window occupies 200x200 (physical) pixels. A 200x200 buffer in the same scenario will just be cropped. This is a good default that makes UI easier to develop, but results in output that looks low res.
I'm currently working around this by setting my buffer to the physical pixel size, then scaling it on application by adding the following to set_buffer in cg.rs:
self.layer.set_contents_scale(width as f64 / self.layer.bounds().size.width);
I'm not sure what the right and real mechanism for doing this should be (e.g. always use physical? allow scale factor choice?)

Softbuffer always uses physical pixels. Your issue is that macOS is scaling the application because it doesn't know it supports HiDPI. You have to find the binary in Finder, Cmd+I on it, and uncheck "Run this app in low-resolution" (or something like that). This should be remembered across builds.

I think that this should not be worked around in the application, softbuffer should apply the contents scale factor to the layer it creates as if the application rendered it by hand. I've made a PR for this.

I think that this should not be worked around in the application, softbuffer should apply the contents scale factor to the layer it creates as if the application rendered it by hand. I've made a PR for this.

Absolutely, I agree.

I pretty strongly thing that the conversion from logical to physical should happen in the consumer of softbuffer, and that softbuffer should always present physical pixels.

I pretty strongly believe that the conversion from logical to physical should happen in the consumer of softbuffer, and that softbuffer should always present physical pixels.

exactly


Disable tokenizers-remote support for the library by default

While convenient, the tokenizers-remote feature means people will have OpenSSL dragged in unless they specify otherwise. It's probably a better default to have it disabled and let people opt in (e.g. the binary) if they want it.

Should gate this behind a feature flag imo 👍🏻


add website packages

@Chenhui-Jia Can you help me double check if the packages (and their versions) are correct/whether I'm missing anything? I just pip show sphinx in the terminal but not sure if I'm missing anything.

@thuntran solved the merge conflict! to avoid more merge conflicts, pull before committing any new changes!😅

@thuntran solved the merge conflict! to avoid more merge conflicts, pull before committing any new changes!😅

Yeah I think I forgot to pull before committing, sorry about that! And thanks for helping with resolving the merge conflict :)

@thuntran solved the merge conflict! to avoid more merge conflicts, pull before committing any new changes!😅

Yeah I think I forgot to pull before committing, sorry about that! And thanks for helping with resolving the merge conflict :) @mariumtapal

@thuntran Oh no you're doing fine - I made a commit to your branch that's why I just mentioned it :)


Improve performance by caching the demangling process

Address #284. Also, refer to RFC PR https://github.com/CppMicroServices/rfcs/pull/2
Signed-off-by: The Mathworks Inc<EMAIL_ADDRESS>

Codecov Report

Merging #286 into development will increase coverage by <.01%.
The diff coverage is 100%.

@@               Coverage Diff               @@
##           development     #286      +/-   ##
===============================================
+ Coverage        84.31%   84.31%   +<.01%
===============================================
  Files               94       94
  Lines             6458     6459       +1
===============================================
+ Hits              5445     5446       +1
  Misses            1013     1013

Impacted Files
Coverage Δ

...mework/include/cppmicroservices/ServiceInterface.h
94.54% <100%> (+0.1%)
:arrow_up:


added buttons to select all tags and deselect all tags

used simple buttons
and integrated using previous functions

Able to select all tags
Able to deselect all tags

fixes #146

made the necessary changes
now using onClick handler
thank you for the review


Wrong Flags

Hello, there's a problem with the country selection. If i select United States, it shows me the Canada Flag. The selection for UK also shows wrong flag.

@NikKovIos  Hi, any news on this issue?

Hello, not yet, sorry. You can try to manage it by yourself and submit a PR then.

@NikKovIos Hi, created PR. https://github.com/NikKovIos/NKVPhonePicker/pull/12

@NikKovIos Hi, created PR.

Thank you. I'll check it asap.

Thanks for your contribution. Merged in https://github.com/NikKovIos/NKVPhonePicker/pull/12


Switch project to absolute imports

switch the project to absolute imports
add an eslint rule to fail on relative imports

How to achieve this:

Follow the docs here: https://create-react-app.dev/docs/importing-a-component/#absolute-imports,
Remove all index.js files
Replace relative imports with absolute imports.

Originally posted by @itsmo-amzn in https://github.com/data-dot-all/dataall/pull/1020#discussion_r1479960349

@petrkalos whats the status here? i see the PR was closed. Can we close this issue or we are considering this for later?

I'd keep it open as it might be a good first issue. I close the PR because it was mostly an automatted change but I didn't have time to test it.


fix: hanging on engine start, if engine starting failed

Looks good. Any tests we can add to test this?

Probably not with the integration tests, only with unit-tests and mock. So far, there are no unit-test for start engine, so I was not able to extend it.


Abstract Simplifier Class

I truly believe the most of the uses if this Library are quite simple and straight-forward, so most of the java boiler-plate can be reduced by extending a class that already overrode the onActivityResult and received the image.
The developer could request an image by calling requestImage(), and receive it in an abstract method called onImagesResult(List). If the developer wants to personalize the selection, he could override a method called customizeImageSelectionParameters(ImagePicker,Builder params) in an Activity. Exemple of utilization:
class MyActivity extends ImagePickActivity{

    void onCreate(/**..
        ...
        ...
        **/
        myButon.setOnClickListener(/** call requestImages() **/
    }

    @Override //These parameteres are Single and return after first by default. Override is optional
    void customizeImageSelectionParameters(ImagePicker.Builder params){
        params.returnAfterFirst(false)
            .multi(true)
            .limit(10);
    }

    @Override //Obligatory override, get your images.
    void onImagesResult(List<Image> images){
        //todo Images selected by user
    }

}

I've created an "ImagePickActivity" and "ImagePickFragment" base classes myself, that uses this library and one that I've made to compress Images. My implementation receives the images onResult, compresses them and calls onImagesResult(). BUT, I've made it in Kotlin. It's simple anyways.
I can share with you and re-implement it in Java if it is an interesting feature to this library.

@vitorhugods it certainly interesting but i have some question regarding extending the library Activity itself

In your use case, why not compressing it after you get the Image object from the library?
Do you have other use cases where it make sense to extending the Activity

Cheers!

I'm indeed compressing it after getting the Image object. My class gets the file provided by the Image Object, compresses it, saves in the app cache folder, and then calls the "onImagesResult" method with the compressed reference.
About the use cases, simplification is the only one that I came across so far. When using the library in several activities I realized that some pieces of code were always the same. So this approach reduced copy/paste.

Did we call the "custom" ImagePickerActivity with Intent API?
TBH i don't want people to extend the Activity especially for a problem / use case that can be handled without extending. So i think this should not be in the library, at least for now.

I considered the option of adding these Activity and Fragment because they are just tools. Users can choose wether to use it or not.
But I see your point, extending an Activity is an aggressive way to reduce copy/pasting.

Will closing this issue for now. Thanks for your suggestion 👍


Only the last pending timeout is cleared during unmount (useTimeoutEffect)

Describe the bug
When multiple timeouts are pending during unmount, useTimeoutEffect only clears the last registered timeout, the other ones stay active.
To Reproduce
Register multiple long lasting timeouts and unmount quickly after registration.
Expected behavior
All timeouts should be cleared upon unmount.
Additional context
Not discovered during runtime but rather by looking at the code itself. One could write a test to verify this issue.

Fixed in df2d247


Cleaning terraform-runner.tls secrets when this is outdated

Hi,
We're using tf-controller with several namespaces, basically one per use case.
I detected the runner is using a different TLS secret each day, one per day.
Let me open this issue in order to add to tf-controller the option/capability to clean the outdated k8s secrets not used.
Let me attach an screenshot of one of our k8s namespaces after 7 days with a several k8s TLS secrets.

Best,
Marc

Have confirmed this with v0.14.1 and resource in 1 namespace

We've found the issue and are working to fix. Essentially it comes down to the GC on these TLS Secrets, the tf-controller currently maintains a list of Secrets that it holds in memory, once the Pod/Container is recreated/restarted that list is gone and Secrets are left around.
We are working to change this so that the list is taken from Kubernetes directly, so on startup any existing TLS Secrets will be cleaned out and going forward for the life of the Pod/Container at renewal time a check will be run that will clean out any TLS Secrets no longer needed. This should hopefully fix this issue and also clean up the issue for anyone who upgrades but has not noticed before hand.

Apologies due to I couldn't answer before. Happy to help and very good job with this fix , it seems the best approach. Thank you!

@mcortinas we have a new image for you to try:
ghcr.io/weaveworks/tf-controller:gc-old-certs-86395477
Please use the rc.yaml and replace image tags to be: gc-old-certs-86395477.


env add results in 'Failed to fetch Amplify Admin credentials'

Note: If your issue/bug is regarding the AWS Amplify Console service, please log it in the
Amplify Console GitHub Issue Tracker

Describe the bug
When adding a new environment the error 'Failed to fetch Amplify Admin credentials' appears.
Amplify CLI Version
4.37.1
To Reproduce
An existing amplify app with a standard 'dev' environment.
amplify env add
Initializing new Amplify CLI version...
Done initializing new version.
Scanning for plugins...
Plugin scan successful
Note: It is recommended to run this command from the root of your app directory
? Do you want to use an existing environment? No
? Enter a name for the environment prod
Using default provider  awscloudformation

For more information on AWS Profiles, see:
https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-profiles.html

Failed to fetch Amplify Admin credentials
init failed
Error: TypeError: Cannot read property 'idToken' of undefined
    at Object.getAwsConfig (/Users/*****/.nvm/versions/node/v11.15.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation/src/configuration-manager.js:761:13)
    at processTicksAndRejections (internal/process/task_queues.js:86:5)

Expected behavior
The expected behavior Is the correct adding of the new environment.
Screenshots
Desktop (please complete the following information):

OS: Mac OS X Big Sur
Node Version. 11.15.0 (also tried on 10.x with the same results)

Closing as a duplicate of https://github.com/aws-amplify/amplify-cli/issues/6037


What happened to the PowerShellAI module?

Existing scripts use the syntax from PowerShellAI module. PSAI is all new cmdlet naming.
What happened?

Sorry, OpenAI has been churning for the last 1.5 years. They changed many things. Including how worked with their backend.
PSAI is my way forward. Even the new things OpenAI dropped last November are being sunset end of the year.
What functions were you calling that are not in PSAI?

The code was for demos used in my AI presentations that highlight Copilot and PowerShellAI.
Basically, I was showing off your module as part of it.

From: Doug Finke @.>
Sent: Wednesday, May 8, 2024 3:25:09 PM
To: dfinke/PSAI @.>
Cc: Mike Nelson @.>; Author @.>
Subject: Re: [dfinke/PSAI] What happened to the PowerShellAI module? (Issue #7)
Sorry, OpenAI has been churning for the last 1.5 years. They changed many things. Including how worked with their backend.
PSAI is my way forward. Even the new things OpenAI dropped last November are being sunset end of the year.
What functions were you calling that are not in PSAI?
—
Reply to this email directly, view it on GitHubhttps://github.com/dfinke/PSAI/issues/7#issuecomment-2101365035, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABPW7UEFEGFE4MTL62BNUUTZBKC2LAVCNFSM6AAAAABHMC5NJ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBRGM3DKMBTGU.
You are receiving this because you authored the thread.Message ID: @.***>


react: transfer types forwardRef

We need to parse the type in the Props and transfer the type to the forwardRef
mitosis input
export interface Props {
  inputRef: MyType;
}

export default function MyComponent(props: Props) {
  return <input ref={props.inputRef} />
}

react output
export interface Props {
  inputRef: MyType;
}

export default forwardRef<MyType>(function MyComponent(props: Props, inputRef) {
  return <input ref={inputRef} />
})

@samijaber we need a parser to parse the types for local exports. We can't transfer types for Angular output either without that

@PatrickJS The path of least resistance to get this working without any additional parsing is to simply access Props["inputRef"]. We know the name of the props type from json.propsTypesRef, and the name of the prop through the getRefsString.
Will put up a WIP PR for you to see what I mean


Calling template tag inside djangocms text ckeditor

Hi!
I would like to know if is it possible to call a template tag inside the text ckeditor, for example:
{% static "mypath/images/next.png" %} inside the text
Thanks

Hello,
Currently it is not possible.
There's no planned effort to make this possible as it opens some security concerns.

@rgcarrasqueira
Maybe give this a try: https://github.com/frnhr/djangocms_renderit
Disclaimers:

I made it
last tested on DjangoCMS 3.1
only use if security is not a concern (i.e. all your users are trustworthy)
only use if security is not a concern, I make no guarantees with regards to safety


Add Clang 3.6 and additional GCC 5 builds to travis

Follow-up to f386742.
(I already squashed the changes into a single commit to make merging easier).

+1
On related note, I get no errors with -fsanitize=undefined with clang-3.7...

@dot-asm you also need to pass -fno-sanitize-recover otherwise the errors are not fatal.

commit'd thanks!

@dot-asm want me to do a PR with adds that flag?

oh, never mind:  that's ghedo talking to andy :)


Image viewer in gall page sometimes adds huge white areas

See image; sometimes but not always (maybe something to do with aspect ratio of the image?) the image viewer (not the modal) is adding huge white areas above and below each image, pushing the source entry way down on the page.

I do not see the whitespace with that species. But the reason that it happens is that one of the images is very tall, it has to scale the viewport to fit the tallest image, otherwise when you click through the images the whole page layout changes and it is very distracting and difficult to use. See for example this one: https://www.gallformers.org/gall/1092 I tried it without the predetemined size and it was much worse. It may be possible to limit the height, but that will require that the images are scaled to fit and they will, on average, be smaller.

That makes sense. Could we pull the text up into the gap on the left when this happens?

I could but it would mean that the text would always be to the left of the images and not flow under it when the images are small. The challenge is that the layout is a grid and there is a limit to what is possible.

Alright sounds like it's not worth addressing at the moment.


submission in "Scoring" status for multiple hours on default queue

I've created a competition at https://www.codabench.org/competitions/1481/ and am trying to test it. However, my submissions either complete within a few minutes (the scoring script takes seconds locally) or sit for multiple hours with the status "Scoring". For example, the submission with ID # 26911 has been in "Scoring" status for more than 2 hours.
Is this a typical amount of time to wait on the default queue, or is something going wrong? If the latter, how can I debug the issue?

Hi,
I think there is something wrong. Sometimes, when the compute worker crashes, the submissions are stuck in "Scoring".
I'll have a look at the status of the workers.

It may not be related, but I copy error logs of one of the workers here to keep a trace:
[2023-10-05 12:15:32,388: INFO/ForkPoolWorker-1] Pull for image: ktgiahieu/codabench-gpu:paper-challenge returned a non-zero exit code! Check if the docker image exists on docker hub.
[2023-10-05 12:15:32,388: INFO/ForkPoolWorker-1] Updating submission @ https://www.codabench.org/api/submissions/26982/ with data = {'type': 'Docker_Image_Pull_Fail', 'error_message': 'Pull for image: ktgiahieu/codabench-gpu:paper-challenge returned a non-zero exit code! Check if the docker image exists on docker hub.', 'secret': 'f36b5e5e-16ea-480c-8661-4518b35a4ef2'}
[2023-10-05 12:15:32,707: INFO/ForkPoolWorker-1] Submission updated successfully!
[2023-10-05 12:15:32,708: INFO/ForkPoolWorker-1] Connecting to wss://www.codabench.org/submission_input/652/26982/f36b5e5e-16ea-480c-8661-4518b35a4ef2/ to send docker image pull error
[2023-10-05 12:15:32,708: INFO/ForkPoolWorker-1] Destroying submission temp dir: /codabench/tmp79wjrq1g
[2023-10-05 12:15:32,746: ERROR/ForkPoolWorker-1] Task compute_worker_run[5c63c4b9-67e7-44a5-a15b-848e8e3970d2] raised unexpected: RuntimeError("can't start new thread")
Traceback (most recent call last):
  File "/compute_worker.py", line 365, in _get_container_image
    container_engine_pull = check_output(cmd)
  File "/usr/local/lib/python3.8/subprocess.py", line 415, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "/usr/local/lib/python3.8/subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['docker', 'pull', 'ktgiahieu/codabench-gpu:paper-challenge']' returned non-zero exit status 2.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/celery/app/trace.py", line 385, in trace_task
    R = retval = fun(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/celery/app/trace.py", line 650, in __protected_call__
    return self.run(*args, **kwargs)
  File "/compute_worker.py", line 115, in run_wrapper
    run.prepare()
  File "/compute_worker.py", line 781, in prepare
    self._get_container_image(self.container_image)
  File "/compute_worker.py", line 378, in _get_container_image
    asyncio.run(self._send_data_through_socket(error_message))
  File "/usr/local/lib/python3.8/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/local/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "/compute_worker.py", line 390, in _send_data_through_socket
    websocket = await websockets.connect(self.websocket_url)
  File "/usr/local/lib/python3.8/site-packages/websockets/client.py", line 535, in __await_impl__
    transport, protocol = await self._create_connection()
  File "/usr/local/lib/python3.8/asyncio/base_events.py", line 986, in create_connection
    infos = await self._ensure_resolved(
  File "/usr/local/lib/python3.8/asyncio/base_events.py", line 1365, in _ensure_resolved
    return await loop.getaddrinfo(host, port, family=family, type=type,
  File "/usr/local/lib/python3.8/asyncio/base_events.py", line 825, in getaddrinfo
    return await self.run_in_executor(
  File "/usr/local/lib/python3.8/asyncio/base_events.py", line 783, in run_in_executor
    executor.submit(func, *args), loop=self)
  File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 188, in submit
    self._adjust_thread_count()
  File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 213, in _adjust_thread_count
    t.start()
  File "/usr/local/lib/python3.8/threading.py", line 852, in start
    _start_new_thread(self._bootstrap, ())
RuntimeError: can't start new thread

Hi, I am having the same (or similar) problem in a challenge our group is organizing (https://www.codabench.org/competitions/1287/): submissions getting stuck with "Scoring" status. Sometimes I just need do cancel the submission and resubmit, but it is a bit inconvenient for participants and organizers. One participant reported that the log was saying that the worker was not able to download the docker image used by the challenge ("Pull for image: spcke/upareval:latest returned a non-zero exit code! Check if the docker image exists on docker hub."). The weird is that it works, but sometimes not. It would be great if you could check what is causing the instability. Thanks

Indeed, I have also observed that the docker pull sometimes fails. We'll investigate it.

Hi, could you find some time to check what is causing the problem? Our challenge, which is approaching its end, has a limited number of submissions at the final phase (set to 3). The inconvenience (for the participants and organizers) is that as organiser I have to manually remove stuck submissions almost every day, so that participants don't loose it. Hope Codabench team can find and solve the problem. I understand it may not be solved for this challenge as we have just a few days left, but would be nice to have this fixed for future challenges.
Best

Hi Julio,
We will investigate the problem. If it is related to the docker pull command, as mentioned earlier, then this problem is fixed in the latest change on compute workers and simply needs to be deployed (we need to update all workers of the default queue manually).

@juliojj I updated the default queue workers. Please tell me if you face the issue again or not.

Hi @Didayolo, it seems the problem persists. I was checking one submission that got stuck, and noticed the output file is generated (based on "output from prediction step"), but the submission gets stuck at the scoring step. I am not sure if it is because of the "docker pull" issue or what. I don't have access to "output from scoring step" as the file seems to be corrupted. Is there anything I can do to help solving this issue?

The ingestion program is not crashing? All logs are normal?

The point is that the same submission file sometimes works, and sometimes doesn't work. The competition finished a few days ago. So, when I have time, I will try to create a copy of the challenge to reproduce the error, and maybe this way we can find and solve the problem. Thanks

We're still seeing this behavior. In many cases, the "Ingestion stderr" of the "Prediction logs" says:
Pull for image: codalab/codalab-legacy:py39 returned a non-zero exit code! Check if the docker image exists on docker hub.

@juliojj @bethard
We found one worker of the default queue that systematically shows this behavior. The other workers seems to be working fine. We just stopped the ill worker so it should work better now.

I am closing this issue to keep track of it only in #1205.
We are going to improve the logs so we better understand the underlying problem of the compute worker when the pull fails / when the submission is stuck to "Scoring".

The problem should be solved one. One compute worker in the queue was ill.


Issue with k8s.io/docs/tasks/debug-application-cluster/debug-service/

This is a Bug Report
Problem:
ISSUE 1: There are two YAML outputs which are displayed as a continuous single line
apiVersion: apps/v1 kind: Deployment metadata: labels: app: hostnames name: hostnames spec: selector: matchLabels: app: hostnames replicas: 3 template: metadata: labels: app: hostnames spec: containers: - name: hostnames image: k8s.gcr.io/serve_hostname

and also
apiVersion: v1 kind: Service metadata: labels: app: hostnames name: hostnames spec: selector: app: hostnames ports: - name: default protocol: TCP port: 80 targetPort: 9376

ISSUE 2: The alpine image suggested in the page kubectl run -it --rm --restart=Never alpine --image=alpine sh seems to have some troubles with nslookup.  It succeeds to resolve nslookup hostnames and the fully qualified name nslookup hostnames.default.svc.cluster.local. Although with some unnecessary warning "server can't find xxx: NXDOMAIN"
/ # cat /etc/resolv.conf
search default.svc.cluster.local svc.cluster.local cluster.local
nameserver <IP_ADDRESS>
options ndots:5

/ # nslookup hostnames
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>:53
** server can't find hostnames.svc.cluster.local: NXDOMAIN
** server can't find hostnames.cluster.local: NXDOMAIN

Name:	hostnames.default.svc.cluster.local
Address: <IP_ADDRESS>

** server can't find hostnames.svc.cluster.local: NXDOMAIN
** server can't find hostnames.cluster.local: NXDOMAIN

/ # nslookup hostnames.default.svc.cluster.local
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>:53

Name:	hostnames.default.svc.cluster.local
Address: <IP_ADDRESS>

However, it FAILS to resolve relative name nslookup hostnames.default
/ # nslookup hostnames.default
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>:53

** server can't find hostnames.default: NXDOMAIN
** server can't find hostnames.default: NXDOMAIN

Proposed Solution:

Display the YAML in correct format

Replace the alpine image by this image to have better net tools (nslookup, curl is also available). All nslookup example works exactly as shown in the page. Including the clean output (without any "server can't find xxx: NXDOMAIN")

$ kubectl run -it --rm --restart=Never minibusy --image=radial/busyboxplus:curl

Please note the radial/busyboxplus:curl image suggested above is used in this Kubernetes doc page Connecting Applications with Services
Page to Update:
https://kubernetes.io/docs/tasks/debug-application-cluster/debug-service/

/kind bug

I think the markup rendering bug was related to issue #20335

This issue was really 2 issues. I've split the remaining piece into #21436.
/close


README.txt doesn't mention manpager.vim

$VIMRUNTIME/plugin/README.txt doesn't include a description of manpager.vim located in the same directory.
Gvim build: gvim_8.1.0189_x86.zip, taken from https://github.com/vim/vim-win32-installer/releases.

README.txt doesn't include a description of manpager.vim located in the same directory.

I'll add a line.
--
hundred-and-one symptoms of being an internet addict:
258. When you want to see your girlfriend, you surf to her homepage.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

closed by https://github.com/vim/vim/commit/a9604e61451707b38fdcb088fbfaeea2b922fef6


Loading empty meshes as render geometry causes a hard crash.

The attached empty.iv file, when loaded in or_rviz, crashes OGRE when used as a render mesh.  It works fine in OpenRAVE's qtcoin viewer.
https://gist.github.com/psigen/19d7fe6d9c8d5c824a3d
[LinkMarker.cpp:323 CreateVisualGeometry] Loaded one or more meshes OpenRAVE because this format is not supported by RViz. This may be slow for large files.

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fffbaed5700 (LWP 5789)]
0x00007fffd380a650 in Ogre::Material::getTechnique(unsigned short) ()
   from /usr/lib/x86_64-linux-gnu/libOgreMain.so.1.8.1
(gdb) bt
#0  0x00007fffd380a650 in Ogre::Material::getTechnique(unsigned short) ()
   from /usr/lib/x86_64-linux-gnu/libOgreMain.so.1.8.1
#1  0x00007fffd3f9f3dc in rviz::InteractiveMarkerControl::addHighlightPass(std::set<Ogre::MaterialPtr, std::less<Ogre::MaterialPtr>, std::allocator<Ogre::MaterialPtr> >) () from /opt/ros/indigo/lib/libdefault_plugin.so
#2  0x00007fffd3f9fc46 in rviz::InteractiveMarkerControl::makeMarkers(visualization_msgs::InteractiveMarkerControl_<std::allocator<void> > const&) ()
   from /opt/ros/indigo/lib/libdefault_plugin.so
#3  0x00007fffd3fa07f2 in rviz::InteractiveMarkerControl::processMessage(visualization_msgs::InteractiveMarkerControl_<std::allocator<void> > const&) ()
   from /opt/ros/indigo/lib/libdefault_plugin.so
#4  0x00007fffd3fa5ba3 in rviz::InteractiveMarker::processMessage(visualization_msgs::InteractiveMarker_<std::allocator<void> > const&) ()
   from /opt/ros/indigo/lib/libdefault_plugin.so

This issue also occurs with empty STL files, such as the following:
https://www.dropbox.com/s/whja8woux0xk4i8/empty.stl?dl=0

I can easily fix this for .iv files. These are not natively supported by RViz, so we load the mesh using OpenRAVE and convert it to a TRIANGLE_LIST type. I can simply skip the conversion if indices is empty. I will add this check.
Unfortunately, this isn't as easy for .stl files. We pass these directly to RViz using the MESH_RESOURCE type without ever loading them in OpenRAVE. We would have to load the model from disk in OpenRAVE just to do this check.
I'm not sure if it's worth it to add the .stl check for a few reasons:

This is a bug in RViz, not or_rviz
We have no way of checking for empty .dae files, which will also most likely crash RViz
OpenRAVE now supports the none geometry type (https://github.com/rdiankov/openrave/pull/306)

The none geometry type is pretty useless for the case of having collision-only trimesh geometry.  It seems to only be useful for links that have no geometry at all.
I agree that fixing this for STLs might be something we have to send upstream.

You're right. none solves the opposite problem (render geometry, but no collision geometry).
My understanding was that qtcoin and or_rviz only rendered visual meshes and did not fall back on rendering primitive objects (e.g. https://github.com/personalrobotics/or_urdf/issues/5). That doesn't seem to the case, so I need to look into that bug. Maybe we can use the same trick to avoid using empty mesh files. :smile:

what happens in RVIZ when the URDF includes an empty .stl file as visual geometry?

@mklingen: I'm not sure, it either loads nothing or falls back on a collision mesh.  I guess it makes sense to try and keep the same logic here.


HR block data request

[ ] nominators
[ ] applicants
[ ] recs
[ ] people who were nominated

first name
email

Andrea
<EMAIL_ADDRESS>

Actually, we won't need an export of this data for h&r block, just export the data to AWS to have a backup


Microsoft Azure best practices

Incidental VM migrations wipe disks; can reduce past quorum
Clock issues (cc @mberhault for more detail)

From @bdarnell in https://github.com/cockroachdb/docs/issues/1132:

One thing we do need to do is add warnings about the data corruption bug that we've seen on Azure A-series instances. D-series instances are believed to be fine; other series should be tested extensively before production use.

The problem on azure is that a wiped disk there comes back with a bad filesystem (ntfs over fuse) which causes severe performance degradation.

Updating this issue:
Warning I proposed in https://github.com/cockroachdb/docs/pull/1638:

On Microsoft Azure, A-series VMs can come back from VM migrations with bad filesystems, which can in turn cause data corruption and performance degradation. We therefore recommend not using A-series VMs. Instead, use D-series VMs, which have not been seen to exhibit these problems. Before using any other Azure VM series in production, you should test them extensively.

@bdarnell's response:

This is not about VM migrations (that issue is common to all azure VMs. It's not data corruption, though, just a performance problem). The issue is that the local disk of A-series VMs just eats data randomly.
Actually, these azure instructions don't specify where the data lives, so they're using the root filesystem. This works (and appears to be immune to the A-series problems), but you have very limited space there. We should update the docs to discuss how to use the local disk (as well as the alternative of using premium storage. Microsoft heavily recommends this and is reluctant to provide support if you're not using it).

@bdarnell, a few follow-up questions so I can refocus this work properly:

Do we still need to say something about the performance implications of VM migrations?
Any guidance you or @mberhault can provide on using local disk or premium storage? That now seems like the main revision needed here.

Yes, I think we do. Users of local disk on azure will need to ensure that their startup scripts examine the partition, and if it's NTFS, reformat it as ext4 (or another linux filesystem). This is kind of a scary thing to do in an automated way, but it's not unique to cockroach - anyone using linux and local disk on azure needs to deal with this.
Microsoft support will tell you that you should be using premium storage for any database; it's hard to get it through their heads that a replicated database can work with non-replicated storage. Premium storage will avoid both the A-series HDD issue and the NTFS issue above, but it's more expensive (and that's going to make azure a more expensive place to run cockroachdb than other providers where we can use local disks).  Our own testing has been almost entirely on local disks so we don't know much about how the performance of the two options compare.

The bulk IO team has found that all azure VMs except the L series suffer from disk throttling that can cause availability issues. So I believe our current recommendation here is to use L-series VMs for production deployments, or D-series VMs as a slightly less expensive alternative for light workloads. A-series are completely busted and should not be used at all; other machine classes have not been tested and should be viewed with suspicion.


Introduce compactScope operation for IfLetStore

As discussed in #496, when using IfLetStore, an action from scoped child store to nil out local state might result in local state being replayed back to earlier value right before the view is removed. The issue can be seen in
https://github.com/pointfreeco/swift-composable-architecture/blob/ba2b9e08e2776bbb02f27ab7123dae767f106fad/Sources/ComposableArchitecture/SwiftUI/IfLetStore.swift#L50-L54
where initial state is copied and used later when the scope transformation returns nil.
A compactScope operation to avoids this by not forwarding non-nil values to the child store.
It's a bit awkward solution, because the implementation is nearly the same as regular scope. I did not want to sacrifice performance of regular scope by having it call compactScope internally.  Maybe a better implementation could be found that could combine both scoping methods.

@ollitapa Thanks for the PR and discussion! This is definitely an annoying pain point, currently. And members of the community have done some good experimentation here. @darrarski wrote about navigation in general awhile ago here, and specifically brought up the issue:
https://github.com/darrarski/darrarski/blob/main/2021/04/Thoughts-on-SwiftUI-navigation/README.md
In it @Thomvis also provides an elegant solution, which I'll reproduce here. You can write a helper transform function that caches the last some case of an optional:
extension Optional {
  static var lastWrappedValue: (Self) -> Self {
    var lastWrapped: Wrapped?
    return {
      lastWrapped = $0 ?? lastWrapped
      return lastWrapped
    }
  }
}

And you can then use this helper when you scope to IfLetStore:
IfLetStore(
  store.scope(state: Optional.lastWrappedValue),
  then: content
)

This should ensure that when you transition away from this view it won't flash blank or even back to the initial value.
We're experimenting with more ergonomic navigation tools to address this issue, but they're not quite there yet. They are, however, powered by this same helper.

Ah, good! That seems a much nicer solution. I added another PR #667 that looks a lot cleaner. Maybe it can be added for the time being, while you work on better tools for navigation :+1:

19 Tem 2021 Pzt 19:16 tarihinde Olli Tapaninen @.***>
şunu yazdı:

As discussed in #496
https://github.com/pointfreeco/swift-composable-architecture/issues/496,
when using IfLetStore, an action from scoped child store to nil out local
state might result in local state being replayed back to earlier value
right before the view is removed. The issue can be seen in
swift-composable-architecture/Sources/ComposableArchitecture/SwiftUI/IfLetStore.swift
https://github.com/pointfreeco/swift-composable-architecture/blob/ba2b9e08e2776bbb02f27ab7123dae767f106fad/Sources/ComposableArchitecture/SwiftUI/IfLetStore.swift#L50-L54
Lines 50 to 54 in ba2b9e0
http:///pointfreeco/swift-composable-architecture/commit/ba2b9e08e2776bbb02f27ab7123dae767f106fad
if let state = viewStore.state {
return ViewBuilder.buildEither(first: ifContent(store.scope(state: { $0 ??
state })))
} else {
return ViewBuilder.buildEither(second: elseContent())
}
where initial state is copied and used later when the scope transformation
returns nil.
A compactScope operation to avoids this by not forwarding non-nil values
to the child store.
It's a bit awkward solution, because the implementation is nearly the same
as regular scope. I did not want to sacrifice performance of regular scope
by having it call compactScope internally. Maybe a better implementation
could be found that could combine both scoping methods.
You can view, comment on, or merge this pull request online at:
https://github.com/pointfreeco/swift-composable-architecture/pull/665
Commit Summary

Introduce compactScope operation for IfLetStore

File Changes

M Sources/ComposableArchitecture/Store.swift
https://github.com/pointfreeco/swift-composable-architecture/pull/665/files#diff-50345ad35b59a32af74fda59695df9c6b1826302d5b41e5d8fbe47477917cc5d
(57)
M Sources/ComposableArchitecture/SwiftUI/IfLetStore.swift
https://github.com/pointfreeco/swift-composable-architecture/pull/665/files#diff-3533da9ef66eaf76e4572f6463cca51759d3fbecff25d79370db849115cc7efa
(4)
M Tests/ComposableArchitectureTests/StoreTests.swift
https://github.com/pointfreeco/swift-composable-architecture/pull/665/files#diff-7ab38c8d80571066ebf95b63685ffdafaa82c427dee15c78672d5576ebe802f6
(62)

Patch Links:

https://github.com/pointfreeco/swift-composable-architecture/pull/665.patch
https://github.com/pointfreeco/swift-composable-architecture/pull/665.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pointfreeco/swift-composable-architecture/pull/665,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATJ5CDLN2QJRFKBH47P2HC3TYRFXHANCNFSM5AUAZAEA
.


Deep Explainer beyond Sequential models

Hi Scott,
Are there any plans to extend the Deep Explainer to Keras' functional api? I'm particularly interested in models with residual connections. Any pointers on if or how this could be achieved would be great. Thanks!

You can pass any pair of input/output tensors to the model, so that should
work for the functional API and also raw tensorflow models. If you run into
any issues with that let me know. The DeepExplainer class is fairly new so
you may run into tensor flow components that are not yet supported, but we
can add.
On Thu, Jul 26, 2018 at 2:57 PM Amir Ghasemi<EMAIL_ADDRESS>wrote:

Hi Scott,
Are there any plans to extend the Deep Explainer to Keras' functional api?
I'm particularly interested in models with residual connections.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/slundberg/shap/issues/182, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADkTxbYkbaD6k3w0_0CzF_C_HOqs4vIqks5uKjs6gaJpZM4Vim9S
.

Thanks, I tried feeding the model's input and output tensors and got this error:
` 142         back_ops = tf.contrib.graph_editor.get_backward_walk_ops(
143             [self.model_output],
--> 144             within_ops_fn=lambda op: op.type not in DeepExplainer.dependence_breakers
145         )
146         self.between_ops = tf.contrib.graph_editor.get_forward_walk_ops(
TypeError: get_backward_walk_ops() got an unexpected keyword argument 'within_ops_fn'`

Thanks for the pointer. I'm getting an error due to tf.pad not being supported.

Thanks for letting me know. I plan to address a bunch of these later this week, so I'll add that to the list.

Support is now on the github version. I'll push a new release with it early next week.


go buttons

The go buttons will look more catchy if on hovering size of go word increases and background colour changes.

@niloysikdar
@nilisha-jais

Assigned @rishika789


sails-sql -f newdb.sql says 'All done' but nothing is generated

I cant see any file in models or controllers...

how did you call the tool? if you call it without pass the -o flag the models will be generated in the directory where the tool was called. You must also have write permission to the directory.

So... after calling:
sails-sql -f b7.sql -o api/models/
a file called undefined.js was generated which contains the next code:
module.exports = {
	attributes: {
	}
};

can you paste the sql file where? so that i can try to generate the error in order to understand the error and solve it.

There you go mate...
b7.sql.zip


HTTP 407 Proxy Authentication Required errors

I cannot use my http proxy with puppeteer-real-browser, it has a username and password along with the host and port.
My crawler works completely fine without the proxy when I put the proxy values in there I get HTTP 407 Proxy Authentication Required errors which tells me my proxy is being set but not being authenticated.
I tried adding this after I define browser, page with my username and password values for the proxy:
        await page.authenticate({             username: <username>,             password: <password>         });
But both with and without that line I get 407 responses when I try to crawl

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Tested with proxy and no problem. Probably the proxy you are using is problematic. You can sign up for webshare and try with the free proxy provided. You can also try puppeteer-page-proxy to solve the problem.

Ok I am using a paid webshare proxy that works in other crawls, but if it works on your end then it must be a problem on my end. Thanks.


Is compatible with php 8.1 ?

Hello, and thanks in advance for your attention on this!
This PHP-Auth is compatible with php 8.1?
I couldn't find the information.
Thanks for taking a moment to read my question!

I've got this working in a PHP8.2 build, should work fine with 8.1 as well.

Thanks for these informations!!


Too long execution time

I have run exactly the same train.sh file with tasks in tune. It takes 5 hours to execute 100 training steps for sst2 on single A100. It doesn't look normal. The only warning is "The NVIDIA driver on your system is too old". Have you encountered this long execution time? Thanks for your help.

It shouldn't be normal. I'll check the code and make appropriate updates in these few days. Have you tried updating the Nvidia driver?

Thanks! I have fixed the problem. Need to upgrade the pytorch-cuda version. :) The version I use now is 11.8.


Blog post: "Pull does not scale - or does it?"

This change is

@grobie @brian-brazil

Great post. Please consider all my comments as nits.
The one thing I really like to stress is missing, though: In a microservice or any multi-tiered architecture, it is fundamentally an advantage if your monitoring system uses the same method to discover targets to monitor as your service instance uses to discover its backends.

@beorn7 That's a great point. I took that sentence and stuck it at the end of the service discovery paragraph.

Thanks a lot for writing this great article!
Given that this debate is already quite a heated one, I'd have tried to use a less lecturing tone in a few places. And I'd suggest to keep the discussion open and using a different final paragraph than telling the reader that they have been wrong. There are likely other concerns people have and we don't know about / you didn't address in the article. Instead of  hardening the fronts further by closing the post with "you're wrong", what about something like "Do you see other scalability issues with the pull model we haven't addressed here? Tell us in the comments!"?

BTW: I don't mind the lecturing tone. On the contrary, I enjoy it being factual. (That's why I suggested to remove the implication the debate of the others is "religious". We would like to make the debate more factual. A lecturing tone might not be the worst strategy to accomplish that.)

I'd like to give this a proper read before publishing. Will get to it in a
few hours.
Agreeing with grobie though that a lecturing tone is not going to convert
anyone who has a strong opinion here. Staying a bit more open doesn't mean
we cannot be factual.
(Haven't read yet to judge the tone myself though.)
On Sun, Jul 24, 2016, 7:46 AM Björn Rabenstein<EMAIL_ADDRESS>wrote:

BTW: I don't mind the lecturing tone. On the contrary, I enjoy it being
factual. (That's why I suggested to remove the implication the debate of
the others is "religious". We would like to make the debate more factual. A
lecturing tone might not be the worst strategy to accomplish that.)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/docs/pull/499#issuecomment-234781161, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEuA8s3Yah-NBR1QvtKQR6bYdOISvdX1ks5qY3rcgaJpZM4JTa12
.

@fabxc I hope I improved the tone already.

Nice read.
I'd probably emphasize the DoS aspect even more. Not only makes our approach Prometheus actually scale well, no, it makes it scale heaps better than classical system where the network is spammed with a ton of events.
Arguably, an issue that can be circumvented in StatsD by pre-aggregating (setup overhead) or by pushing metric states in intervals. But there's no system out there that does it, right?

I would assume that most metrics-based push-driven monitoring systems don't suffer from the StatsD "one packet per event" fallacy. I think we shouldn't speculate too much about other systems but focus on what's the case in the Prometheus ecosystem, which in fact cannot suffer from random monitored clients deciding to send data to the monitoring server. Also, Prometheus has several protection mechanisms against run-away scrapes and too many samples. Those could be improved' and a push system can implement protection mechanisms, too, but I'd say we can safely claim that the protection is easier to get in a pull-based system.
WRT convincing those with a strong opinion: I believe we won't convert many of those anyway, and luckily, I don't think there are so many out there in the first place. Our most important audience are those deciding which monitoring system to use, and those should not be scared away by unfounded claims that pull doesn't scale. For those, a nice factual and concise explanation is best.

Sounds good, thanks 👍

LGTM :+1:

Thanks everyone, that was more feedback from more people than I had expected :) I will get to all the points in detail in the next days.

Hey, can we publish this soon. I just had a meeting with an analyst where "pull doesn't scale by definition" came up again.

This is :+1: from my side. :ship:

Yep - I should finally find time to respond to the last comments and then
merge it later today or tomorrow :)
On Thu, Aug 4, 2016 at 3:04 PM, Björn Rabenstein<EMAIL_ADDRESS>wrote:

This is 👍 from my side. 🚢
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/docs/pull/499#issuecomment-237546325, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAg1mOL_wQ1mD9WZkaAMoHpbf7TYXJoHks5qceNTgaJpZM4JTa12
.

Addressed some comments and ignored some others :) Gonna post now...


Federico: Watch solutionbyyouth.com

Federico requests the watch of the watch_keyword solutionbyyouth\.com. See the MS search here and the Stack Exchange search here.

!!/approve

!!/approve

solutionbyyouth\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.


Make IsXXXBasedEra a class hierarchy

Changelog
- description: |
   Make `IsXXXBasedEra` a class hierarchy
  type:
  - compatible

Context
The PR makes IsXXXBasedEra a class hierarchy of one era being based of the next. I was surprised it's not the case in the code; I need to create a lot of dummy proofs like
maryFromAlonzo :: IsAlonzoBasedEra era => (IsMaryBasedEra era => a) -> a

in my projects.
How to trust this PR
It's a QOL improvement, just explicitly states a true fact of eras forming a chain.
Checklist

[x] Commit sequence broadly makes sense and commits have useful messages
[ ] New tests are added if needed and existing tests are updated. See Running tests for more details
[x] Self-reviewed the diff

@carbolymer can you reapprove? I had an unverified commit, should be good now.


adding proxy auth to pypub

Adding proxy authentcation to pypub

Hey @imakelists, thanks for your interest in the project.
Unfortunately switching from urllib.request to requests is not a simple switch, and using urllib.request was very intentional since it supports additional schemes like file:// which may sometimes be required whilst requests does not support this out of the box.
Additionally, urllib already supports http-proxies according to the existing documentation: https://docs.python.org/3.5/howto/urllib2.html#proxies
In fact, you can pass proxy settings via env-variable using HTTP_PROXY with the included authentication and it should be automatically parsed and included in the associated requests.
If you'd still like to do it programatically I'd ask you to use urllib's  ProxyHandler if possible please. Lmk if you have any questions or concerns. Thanks :)


System privileges editor has incorrect URL prefix for Intrusion Detection log file browser

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
System privileges editor has incorrect URL prefix for log file browser in Services: Intrusion Detection:

/ui/diagnostics/log/suricata/*
/api/diagnostics/log/suricata/*

Correct paths for Intrusion Detection (suricata) log viewer URL are:

/ui/diagnostics/log/core/suricata/*
/api/diagnostics/log/core/suricata/*

This causes Intrusion Detection -privileged  user account to have access to settings but not log file browser.
To Reproduce
Steps to reproduce the behavior:

Edit limited users or groups system privileges
Enable Services: Intrusion Detection access for the user
Login as limited user
Services -> Intrusion detection -> Log File menu item is not available

Expected behavior
User should have access to  Services -> Intrusion detection -> Log File and related /ui/diagnostics/log/core/suricata URL path.
Environment
OPNsense 22.1.7_1-amd64

Relevant privilege editor UI displaying the incorrect URL prefix for Log File menu item:

@imyller nice catch, thanks! should be fixed in https://github.com/opnsense/core/commit/e810712f8b27068a4fec4b3ba0822e783851977a


Boolean fields not showing up in embedded Mongoid form

I have two Mongoid based models. One is called User. The other is called NflAuthorization. I'm simply trying to get optional check boxes that default to false to show up in an edit portion of Rails admin to toggle the values between true and false. This seems to come up empty with Rails Admin as-is. Is this a current issue or am I just not handling it right? Here's a gist of my code:
https://gist.github.com/stevef/53da620f8d996e22259d

Should have been solved by c262d4080819084e02af7860d7ffd546c4b6e532.


Fix text readability, change from hairline/light/thin to regular/bold

Issue: #57
Specifically the part

A lot of the text is too hard to read because it its thinness and bad contrast.

Requirements

[ ]  issue was opened to discuss proposed changes before starting implementation.

I didn’t specifically open an issue, but the existing #57 already mentioned the problem.

[x]  during development, node version specified in package.json was used.
[x]  package versions and package-lock.json were not changed (npm install --no-save).
[x]  app version number was not changed.
[ ]  all new code has tests to ensure against regressions.

Wasn’t sure how to best test loop through all the breaks and microbreaks. The indicator icon to get to the settings was not shown in development (am on Ubuntu).

[x] npm run lint reports no offenses.
[x] npm run test is error-free.
[ ]  README and CHANGELOG were updated accordingly.

Didn’t take new screenshots of all the pages.

Description of the Change
The hairline, thin and light font weights make the text very difficult to read and result in subpar contrast on basically on the colors. You should be able to read the break suggestions quickly so you can perform them, and not harm your eyes further.
That’s why the font weights were replaced with higher weights from the same font, Lato:

Hairline to Regular
Thin and Light to Bold (also to not have too many different weights which are almost the same anyway)

I downloaded Lato from Google Fonts and it only included the ttf. If all the other formats are indeed needed, let me know where you downloaded them from and I’ll add the files for Regular and Bold.
Verification Process
I checked out the initial run window and it looked good. Can’t make it show up again though to take screenshots, and can’t loop through the break suggestions or show the settings window as mentioned above. :\ Let me know how it works and I’ll continue with that. :)

The Appveyor failure seems to be unrelated to this change as only CSS and font files were touched? https://ci.appveyor.com/project/hovancik/stretchly/build/1.0.1021/job/qiqjt37x16ai3xrs

Yeah, appveyor seems to fails tests on random lately.
You might need https://github.com/hovancik/stretchly#linux-note to get tray working correctly on Linux.
Here is what I am getting on Linux with current version:

and on mac:

and with your change it looks like this (on Linux):

so this is something platform specific... Maybe some font related CSS? I'm not sure what mac/linux/windows are doing differently when rendering fonts, but I think that's where solution might be. What do you think?

this is mac:

I would definitely say it’s way easier to read. Yes, on macOS it seems less bad in the current version than on Linux (which I am using), but still thin fonts are not ideal.
I would actually do additional things like:

Bold the heading (regular for body text) for better separation – this relates a lot with this pull request
Possibly cut down the text and improve wording of each break so there’s less to read
Making the body text left-aligned for even more readability
Introduce a symbol (maybe emoji) to show for each activity to even more quickly see what it’s about

And all of these would change the look quite a lot – but it’s best to do stuff step-by-step so it’s easier to review and discuss. So yes, I definitely think that this pull request is a step in the right direction, but it’s only the first one.
What do you think?

I've created #304 and tested on win, amc, debian. Thanks for bringing this issue forward ;]


Replace unsafe dynamic $_SERVER['SERVER_NAME'] by safer YOURLS_SITE constant in login redirect

Issue when webserver serving YOURLS is set with default/empty SERVER_NAME (e.g. _ in nginx) and server name is set by reverse proxy in front.
Security issue with some overridable values : https://expressionengine.com/blog/http-host-and-server-name-security-issues
Using defined constant is safer
Handles scheme, port and in URL username & password

Although I agree that $_SERVER['SERVER_NAME'] can cause problems in an improperly configured server, the YOURLS_SITE constant currently also causes problems for newbies.
A better replacement might be $_SERVER['HTTP_HOST']as this depends on the request from the client.
I would vote to reject this PR as written.  After my extended health absence, I have started to write the PR and Plugin to solve ALL the Protocol Problem (http|s) and I believe it will solve this issue also.
Likewise, the cpnstants YOURLS_SITE and YOURLS_ADMIN_SSL will be removed from the core in this coming PR to prepare for a fast and easy Full-Fledged Graphical Installer and a true YOURLS Multisite (one set of files and more than one domain with different keywords per domain).  I expect the Protocol Problem PR and HTTPS Plugin will be complete and tested by December. (I currently have an Alpha Prototype under test and am refining the code.)

Any news ????

I was side tracked by a stroke last year.
SERVER_NAME is only unsafe due to bad coding use, not the element itself.
The real way forward is to remove all core constants except for the
database entries. Starting with the YOURLS_SITE constant.
This should be split into two database entries (one to build short URLs,
and one to build admin urls). These should be modified by a plugin for
people who do not need the default settings. I had this 95% coded when I
had my stroke.
On Wed, May 8, 2019, 18:46 ymage<EMAIL_ADDRESS>wrote:

Any news ????
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/YOURLS/YOURLS/pull/2442#issuecomment-490439325, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHYYPQGYT4WO4N77RV7MNLDPUKVQHANCNFSM4FW54N3A
.

Hi,
Sorry to hear that.
Hope you recovered well.


Failed to parse abi: unsupported arg type: contract IPositionMetadata

Can be invoked by attempting to parse ABI for: 0x4A3e49f77a2A5b60682a2D6B8899C7c5211EB646

Resolved in #140. Needs merge.


Made socket listener optional (required when using dev-middleware only)

This PR makes the socked client optional which was used before for some more detailed error reporting. Unfortunately, this socket is only available when using webpack-dev-server and not webpack-dev-middleware.
The PR splits the entries for a basic integration and the socket listener into two. It then only adds the socket listener entry whenever there is a devServer config entry in Webpack.
The PR restores the previous behavior of v0.16.x for non-dev-server users.

Thanks!

Is this a breaking change? It doesn't really look like one.

No it is not.


menu should disappear when certain items are clicked

For instance, when clicking File-->Open or File-->Save, the menu should disappear after clicking the option.
It should stay visible for others, such as anything under View, since a user might want to check many of those at once.

Save already disappears after clicking. So the only buttons I identified that do not close the menu is File --> Open
File --> Import cadnano v2
View --> anything
So, I can make it so that the two File buttons will make menu disappear, but leave view alone.


Fixes #803: "make clean" no longer removes ramda.min.js

make clean; make doesn't rebuild ramda.min.js so git status marks
it as deleted.
@davidchambers says ramda.min.js is only updated when a new version is
published, so let's not remove it on clean.

:deciduous_tree: Very nice. Thanks, Nic!


docs: Improved example search

Convert to simplified examples repo. Goal here is whenever I create an example for someone, I'll push it to https://github.com/wxt-dev/examples and it will show up in the docs automatically.
I'm not merging this until I move all the examples over to the new repo.

@davestewart I would love your option on this new search, if it's more helpful than the last.

New: https://deploy-preview-751--creative-fairy-df92c4.netlify.app/examples.html
Old: https://wxt.dev/examples.html

Hey,
Interesting idea! Not sure how much value the search offers, though I do like the idea of a separate repo. Interestingly I just checked Chrome's examples, and they do the same!
I think a left-hand filter would work better. There's certainly enough space on the page to do that as well. Also wrapping checkboxes with labels would make for better UX.
FWIW, Nuxt and its Content module, along with server/api routes is another option to do search, vs a script + JSON update. Might be worth considering in the future.

@davestewart It is based off Chrome's new examples! I really like how to you can quickly find examples that use a specific API, and see the "right" way to use them. I wanted to add the packages filter as well, so people can find examples using different NPM packages.
I can clean up the CSS and move it to the side, just wanted to get a POC out for some feedback.

@davestewart Hows this?

Also wrapping checkboxes with labels would make for better UX.

Not really sure what you meant by this, I was already wrapping the labels. This time I tried not wrapping (you can scroll horizontally)

Bedtime here; will take a look first thing!


docs: correct stream method documentation

The documentation of the stream method had several issues. orderBy came before stream and orderBy is in reality order and one execute was missing.

Should I fix those lint errors, or should we switch to the recommended package lints? https://dart.dev/guides/language/analysis-options#lints

@Vinzent03
Thank you so much for spotting this error. It would be great if we could get the lint errors fixed even though it's not directly related to this PR itself!

I've chosen the wrong branch. See #81 now


[5.6] Fix assertSessionHasErrors

Here the variable used with is_int should be $key no $value (same as in Laravel 5.5).

Arg... Had my pull request ready!

@XavRsl haha, that has happened to me before, I double checked, but still knew someone else was probably going to send the same fix soon. This speaks very well about the Laravel community, I guess :)


[Issue]: Cannot find node after install lts and use lts

What happened?
Cannot find node command.
I didn't install node before.
After I installed nvm and nvm install lts, there is still no node in my terminal.
I use Terminal Application, and run a Powershell 7.2.0-preview.2 version
There is no error in nvm list and nvm use lts.
What did you expect to happen?
I want use node
Version
1.1.11 or newer (Default)
Which version of Windows?
Windows 11+
Which locale?
Asian
Which shell are you running NVM4W in?
PowerShell
User Permissions?
Administrative Privileges, Elevated
Is Developer Mode enabled?
Yes
Relevant log/console output
PS C:\Users\sf235> node
node: The term 'node' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Suggestion [4,General]: The most similar commands are: move, ndr, cd, md, popd, copy, del, gdr, nal, ni.
PS C:\Users\sf235> nvm list

    18.16.0
PS C:\Users\sf235> nvm use lts
Now using node v18.16.0 (64-bit)
PS C:\Users\sf235> node
node: The term 'node' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Suggestion [4,General]: The most similar commands are: move, ndr, cd, md, popd, copy, del, gdr, nal, ni.
PS C:\Users\sf235> node -v
node: The term 'node' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Debug Output
Running NVM for Windows with administrator privileges.

Administrator: PowerShell
NVM4W Version:      1.1.11
NVM4W Path:         D:\Develop\nvm\nvm.exe
NVM4W Settings:     D:\Develop\nvm\settings.txt
NVM_HOME:           D:\Develop\nvm
NVM_SYMLINK:        D:\Develop\nvm
Node Installations: D:\Develop\nvm

Active Node.js Version: none
(run "nvm use <version>" to activate a version)

No problems detected.

Find help at https://github.com/coreybutler/nvm-windows/wiki/Common-Issues

Anything else?
No response

The NVM_SYMLINK after install is D:\Develop. I change it to D:\Develop\nvm for test but nothing changes.

Typically the NVM_HOME and NVM_SYMLINK values are different. For example:

What happens when you run D:\Develop\nvm\v18.16.0\node.exe -v?

Typically the NVM_HOME and NVM_SYMLINK values are different. For example:

What happens when you run `D:\Develop\nvm\v18.16.0\node.exe -v`?

Thank you for your reply. It shows node version v18.16.0 correctly.
I change NVM_LINK to D:\Develop again and nothing changes.
BTW, I didn't see any symbol link in nvm folder or its parent folder. I mean node.lnk.
Maybe this is the point?
PS D:\Develop\nvm> ls

    Directory: D:\Develop\nvm

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
d----           2023/5/13    13:45                v18.16.0
-a---           2021/12/7    12:36            305 elevate.cmd
-a---           2021/12/7    12:36            335 elevate.vbs
-a---           2023/1/26    21:28            724 install.cmd
-a---           2021/12/7    12:36          15086 nodejs.ico
-a---           2023/4/12    22:30        8294304 nvm.exe
-a---           2023/5/13    12:10             40 settings.txt
-a---           2023/2/13    20:54            384 setuserenv.vbs
-a---           2023/5/13    12:10          18797 unins000.dat
-a---           2023/5/13    12:10        3048509 unins000.exe
-a---           2023/2/13    20:54            286 unsetuserenv.vbs

Did you mean NVM_SYMLINK? NVM_LINK is not recognized by NVM4W. There is never a node.lnk file.
Again, run D:\Develop\nvm\v18.16.0\node.exe -v. It should output the version. If this works, then there is a different path problem.
Also, is your D drive a mapped directory on a network drive or does it reference a hard drive/partition?

Q1: Yes, I mean NVM_SYMLINK. And node.lnk is just something I imagine. Sorry for the bother.
Q2: And yes, the D:\Develop\nvm\v18.16.0\node.exe -v print the node version correctly.
Q3: My D drive is just a HDD in my pc.

I must be tired lately, because I should have seen the problem straight away.
The NVM_SYMLINK cannot be an existing real directory. Every time nvm use is run, it attempts to update the NVM_SYMLINK. Since this is a real directory in your environment, there is no symlink to change. As a result, Windows thinks node.exe exists directly under D:\Develop\nvm instead of D:\Develop\nvm\v18.16.0. Bottom line: nvm use isn't doing anything when NVM_SYMLINK already exists as a real directory.
You mentioned that NVM_SYMLINK was originally D:\Develop. Since this is a real directory too, the problem would be the same. Changing it to D:\Develop\nvm changed it from one real directory to another. If you want to keep everything isolated in the D:\Develop\nvm directory, then I'd recommend changing NVM_SYMLINK to D:\Develop\nvm\installations. The installations directory should not exist when you do this.

Thank you sir! It's all about that NVM_SYMLINK original value.
So why it was D:\Develop after my installation?


I can't find my key/value set by phpfastcache with ssdb driver

Hello,
I set key / value in cache with phpfastcache and ssdb driver, with php i can see the key is been in cache i can get the value but when i search with phpssdbadmin or command line i can't find the key / value
I don't understand why ?
Thanks

check your php code, maybe you put some unseen characters in the key.


Feature/embeddable charts

What kind of change does this PR introduce?
Fixes https://github.com/supabase/repository.surf/issues/1
Allows iframe embedding of our charts, demo: https://codepen.io/joshenlim/pen/oNzpeLZ?editors=1000
Done via this url: http://localhost:3000/supabase/supabase?embed=true&type=[chartType]
Where chart type is currently either issues or stars

I couldnt see this one working from the code pen -

Perhaps it's not deployed yet though?

Oh that's weird hmm, could you try it in incog? This is how it's looking like for me

Tried again in incognito (Firefox) with adblock turned off - same result.

Gotcha! I'll reopen the issue again - will investigate

Perhaps also just embed the chart, but not the title & text etc? People might want to add their own text/language

Ahhh okay, so there was something going on with using iframes in codepen on Firefox specifically, here's a demo on codesandbox instead: https://codesandbox.io/s/supabase-embed-demo-iyzyk
Will keep the issue opened - still want to do some refactors as well


Add the helplink for Image Annotation

Is your feature request related to a problem?
The docs for Image Annotation are present in the circuitVerse docs but its helplinks are not available in the simulator.
Describe the solution you'd like
Like other Elements, we need to add the helplink for the Image Annotation.
As we can see the help button in properties section

The help link button is not present in Image Annotation.

Are you working on this? (Yes/No)
yes

Can I help up with the issue please?


TensorFlow device scopes and shared weights

I'm trying to understand how shared layers work with the functional API as described here, particularly in conjunction with TensorFlow device scopes as described here, and whether this can be used as a (albeit inefficient) method for multi-device training.
I understand that I can specify a model with shared weights by first defining the model itself on one device:
with tf.device('/cpu:0'):
    sub_x = Input(shape=(...,))
    sub_y = <define model layers here>
    sub_model = Model(input=sub_x, output=sub_y)

And then call sub_model repeatedly to produce replicas on other devices, which can then be fed separate data, then have their output activations merged and a loss computed for training:
with tf.device('/gpu:0'):
    a1 = sub_model(data_1) # all ops in the replica will live on GPU:0
with tf.device('/gpu:1'):
    a2 = sub_model(data_2)
with tf.device('/gpu:2'):
    a3 = sub_model(data_3)
with tf.device('/cpu:0'):
    merged = merge([a1, a2, a3], mode='concat', concat_axis=0)
model = Model(input=x, output=merged)
model.compile(optimizer='sgd',
              loss='mse')

How does Keras ensure the weights of the individual replicas are kept in sync, despite each being fed separate data? I'd guess the mechanism is much more inefficient than simply averaging parameter updates using TensorFlow server objects What's more, here you suggest this approach is the only way to do data-parallalism in Keras @fchollet
This seems to work on small dummy networks, but possibly doesn't scale? This would only work if the parameters on each device were kept synchronised constantly, which sounds as if it might be expensive if implemented naively... when different updates are made to each of the model replicas, are the weights guaranteed to be kept up to date? Rather than relying on this, should I be using a TensorFlow optimizer and averaging and applying gradients specifically instead?
Is the implementation in the TensorFlow backend perhaps using a similar strategy to the following TensorFlow example, where tf.variable_scope(tf.get_variable_scope(), reuse=reuse_variables) is used? The blogpost on using TensorFlow with Keras seems to suggest that both serve the same role, it's written: "Variable sharing should be done via calling a same Keras layer (or model) instance multiple times, NOT via TensorFlow variable scopes" but is the underlying mechanism also very similar?

Anyone have tried this multi gpu way?


Use Convex for trelleux example database

Hi! I'm the autofix.ci troubleshooter bot.
It looks like you correctly set up a CI job that uses the autofix.ci GitHub Action, but the autofix.ci GitHub App has not been installed for this repository. This means that autofix.ci unfortunately does not have the permissions to fix this pull request. If you are the repository owner, please install the app and then restart the CI workflow! 😃


unraid error Migration table is already locked

i have an error when i try to start the proxy manager in unraid. it gives me: error Migration table is already locked. info Current database version:<PHONE_NUMBER>2346. Can't take lock to run migrations: Migration table is already locked
If you are sure migrations are not running you can release the lock manually by deleting all the rows from migrations lock table: migrations_lock.
it worked fine until i restarted the server. how do i go about fixing this? i dont have mariadb installed in docker.

Hi,
Can you please provide the whole container log?
Thanks!

Hi,
Can you please provide the whole container log?
Thanks!

https://pastebin.com/eTf1supx

Can you please try the latest image?  It should fix the issue.

Closing the issue.  Please reopen if needed.


ManyToMany relationships

Hey @tabuna :) I am keep working on and writing about Orchid CRUD. I was wandering, what about ManyToMany relationships?
Imagine I have two resources:

Movie
Genre

and I want to be able to define attach genres to a movie. How would you proceed? Is this something in the plans?
Thanks a lot for your precious time

I am using "Relation" as a field, but I am afraid isn't enough right now for the CRUD module

I am using "Relation" as a field, but I am afraid isn't enough right now for the CRUD module

Hey @francescomalatesta
What do you mean by not enough? Can you describe what you would like to see?

Hey @francescomalatesta
What do you mean by not enough? Can you describe what you would like to see?

Sorry, you're right. I should add more data.
So, as I said before, I have two models:

Movie
Genre

I defined the many to many relationship in the models. Then, I defined the relationship field in the MovieResource like
Relation::make('genres.')
                ->fromModel(Genre::class, 'name')
                ->multiple()
                ->title('Genres')

like I saw here: https://orchid.software/en/docs/field/#relation
When I try to create a new movie and assign a single genre, it says
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'genres' in 'field list' (SQL: update `movies` set `genres` = ["1"], `movies`.`updated_at` = 2021-01-15 22:25:57 where `id` = 2)

If I try to assign two genres, the error is
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'genres' in 'field list' (SQL: update `movies` set `genres` = ["1","2","3"], `movies`.`updated_at` = 2021-01-15 22:27:20 where `id` = 2)

Instead of trying to write on the pivot table, my guess is the package is trying to assign those values to a "genres" field that doesn't exist

Sorry, you're right. I should add more data.
So, as I said before, I have two models:

Movie
Genre

I defined the many to many relationship in the models. Then, I defined the relationship field in the MovieResource like
Relation::make('genres.')
                ->fromModel(Genre::class, 'name')
                ->multiple()
                ->title('Genres')

like I saw here: https://orchid.software/en/docs/field/#relation
When I try to create a new movie and assign a single genre, it says
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'genres' in 'field list' (SQL: update `movies` set `genres` = ["1"], `movies`.`updated_at` = 2021-01-15 22:25:57 where `id` = 2)

If I try to assign two genres, the error is
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'genres' in 'field list' (SQL: update `movies` set `genres` = ["1","2","3"], `movies`.`updated_at` = 2021-01-15 22:27:20 where `id` = 2)

Instead of trying to write on the pivot table, my guess is the package is trying to assign those values to a "genres" field that doesn't exist

Let me know if you need anything else that can help!

Let me know if you need anything else that can help!

The area of responsibility of the fields is much smaller than you might be used to. They do not, in any way, affect the behavior of the database or models. In reality, the error occurs only because there is no processing of this connection.
Resource Events
Each resource has two methods that do the processing, onSave and onDelete. Each of them is launched when the event is executed, and you can change or supplement the logic:
/**
 * Action to create and update the model
 *
 * @param ResourceRequest $request
 * @param Model           $model
 */
public function onSave(ResourceRequest $request, Model $model)
{
    $model->forceFill($request->all())->save();
}

/**
 * Action to delete a model
 *
 * @param Model $model
 *
 * @throws Exception
 */
public function onDelete(Model $model)
{
    $model->delete();
}

The area of responsibility of the fields is much smaller than you might be used to. They do not, in any way, affect the behavior of the database or models. In reality, the error occurs only because there is no processing of this connection.
Resource Events
Each resource has two methods that do the processing, onSave and onDelete. Each of them is launched when the event is executed, and you can change or supplement the logic:
/**
 * Action to create and update the model
 *
 * @param ResourceRequest $request
 * @param Model           $model
 */
public function onSave(ResourceRequest $request, Model $model)
{
    $model->forceFill($request->all())->save();
}

/**
 * Action to delete a model
 *
 * @param Model $model
 *
 * @throws Exception
 */
public function onDelete(Model $model)
{
    $model->delete();
}

Super clear! Thanks and sorry if I am making a lot of questions. The more I use orchid the more I love it, so I want to switch my mentality from Nova to this one.
Have a nice day.

Super clear! Thanks and sorry if I am making a lot of questions. The more I use orchid the more I love it, so I want to switch my mentality from Nova to this one.
Have a nice day.

thx thx thx, so very much


Bug: TextView doesn't work on Windows/.NET Core -> crashes almost immediately

using Terminal.Gui;

class Program
{
	static void Main()
	{
		Application.Init();
		var top = Application.Top;

		var win = new Window("App")
		{
			X=0,
			Y=1,
			Width=Dim.Fill(),
			Height=Dim.Fill()
		};
		top.Add(win);

		var textView = new TextView()
		{
			X=0,
			Y=2,
			Width=40,
			Height=40
		};

		win.Add(textView);

		Application.Run();
	}
}

This code causes the app to crash as soon as you clicking in the console or want to type a character.
Error is:

->Application.Run(): System.ArgumentOutOfRangeException: "Index was out of range. Must be non-negative and less than the size of the collection."

I could not reproduce this on Unix.
Would you mind trying this from Visual Studio and finding out where it is crashing?

It is crashing when calling the Run method:
`// Zusammenfassung:
//     Runs the application with the built-in toplevel view
public static void Run();`

Stack trace:
System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
  at at System.Collections.Generic.List`1.get_Item(Int32 index)
  at at Terminal.Gui.TextModel.GetLine(Int32 line)
  at at Terminal.Gui.TextView.GetCurrentLine()
  at at Terminal.Gui.TextView.MouseEvent(MouseEvent ev)
  at at Terminal.Gui.Application.ProcessMouseEvent(MouseEvent me)
  at at Terminal.Gui.WindowsDriver.Mono.Terminal.IMainLoopDriver.MainIteration()
  at at Mono.Terminal.MainLoop.MainIteration()
  at at Terminal.Gui.Application.RunLoop(RunState state, Boolean wait)
  at at Terminal.Gui.Application.Run(Toplevel view)
  at at Terminal.Gui.Application.Run()
  at TestGuiCs.Program.Main(String[] args) in D:\Test-Projekte\TestGuiCs\TestGuiCs\Program.cs:33

GetLine throws an IndexOutOfRange exception when trying to get a line when no text has been set on the control. The lines collection is empty. Setting the Text property of the control to an empty string when instantiating the control works around the issue

No repro. Closing.


#1472 - Icon slide animation inside the button

Issue:
https://github.com/sButtons/sbuttons/issues/1472
I have added a new animation to the category Icon buttons.

Awesome work thank you!
@all-contributors please add @naveensunkara for code and idea


Favorite button causes navigation on mobile

Describe the bug
If you favorite a server on mobile it causes navigation to the server details page.
To Reproduce
Steps to reproduce the behavior:

Make sure the mobile layout is used (cards)
Favorite any server on the homepage.

Expected behavior
Favoriting a server on mobile shouldn't cause navigation, just like on desktop.

Hey! i fixed the issue and made a pull request, can you review it?

Hey! i fixed the issue and made a pull request, can you review it?

Thank you for fixing this!

Thank you for fixing this!


Add sd_hash to payload of Key Biniding JWT example

According to spec Key Binding for SD-JWT-VC,

the Key Binding JWT MUST adhere to the rules defined in Section 5.3 of SD-JWT spec

This means, that the sd_hash claim has to be included in the payload of the Key Binding JWT. Can you correct the example, given in SD-JWT-VC with Key Binding example, so that it includes the sd_hash claim?

Good catch! We will fix this. Indeed, the sd_hash is missing.

@danielfett can you fix the example?

Looks like we need to switch back to the pointing at the main branch of the sd-jwt-python library for generating the examples. sd_hash should show up automagically once that's done.


fix: azure sdk httpclient instrumentation issue

Description
Fixes ASSETS-4291
node-openwhisk-newrelic was having issues when used with aio-lib-state
was patched by commenting out this line: https://github.com/adobe/node-openwhisk-newrelic/blob/11c06eb702aadd31886b44ab731f34e5f010ffad/lib/probe/http-client.js#L250
Fixes # (issue)
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@jdelbick I don't have the required permissions to do that. I can do that if you add me as a contributor or smt.

@jdelbick I got my permissions sorted out. Created a new PR https://github.com/adobe/node-openwhisk-newrelic/pull/57


KeyError: 'size' from fetch_working_orders()

Not 100% sure but I believe Version 2 now refers to order size as "orderSize" not "size" so the above error is occurring when fetching working orders.

Actually, there is more work to do here. Looking at the code, it uses a mix of version 1 and version 2 calls. It would be better to set the version globally and then make the correct call based on the global version. Better still, it would be better to create a release branch for Version 2 which only refers to version 2.
In this case, fetch_working_orders() which makes a call to /working orders needs to be updated to reflect the API reference here: https://labs.ig.com/rest-trading-api-reference/service-detail?id=555
I did that locally, needed to make a minor change to my implementation of the library and that fixed all errors relating to the differences between version 1 and 2.
For now, I'm going to set it to call version 1 as most of the rest of the code does. I'll come back later to updating the code to use version 2 which I'm keen to do as the naming is a lot clearer.


from upstream

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 15 committers have signed the CLA.:white_check_mark: wenfengwang:x: Michael Gasch:x: n3wscott:x: smilence-yu:x: inigohu:x: muncus:x: igolaizola:x: sasha-tkachev:x: lionelvillard:x: duglin:x: matzew:x: dependabot[bot]:x: sforests:x: nbajaj90:x: graysideMichael Gasch seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Adding RSS feed file

Blog readers like Thunderbird need a feed.xml file so that they can keep track of new posts.

Thanks, Greg!


Issue-6420 Add support for Apache Datasketches CPC sketch

The CPC sketch is a space-efficient cardinality estimation algorithm. More detail can be found on the Apache Datasketches website documentation: https://datasketches.apache.org
Support is added here for the following:

ingestion via transformation function
extracting estimates via query aggregation functions
segment rollup aggregation
StarTree aggregation

I have tested the following features on this PR:

ingestion transformation function using baseball stats
segment rollup aggregation with MergeRollupTask
StarTree aggregation
query aggregation

Accumulating results on a server into a list before merging yielded marginal performance gains at the cost of code complexity and this approach was abandoned.
Some ideas for future improvement are:

extensible aggregation functions via plugin system
adding sub-packages to the existing aggregation functions and factoring out common parts.

For the second point, a base class has been used in some instances and a similar approach could be used with the sketch implementations - with the exception of the Theta Sketch.

Thanks for the contribution! Can you help also add the documentation for this new function?

Thanks for the contribution! Can you help also add the documentation for this new function?

Yes, I'll try and update the documentation for our earlier contributions as well.

Thanks for the contribution! Can you help also add the documentation for this new function?

Hi @Jackie-Jiang I have prepared some documentation here for review:
https://github.com/pinot-contrib/pinot-docs/pull/254


Rename from UOR to Emporous

Need a full recursive grep / rename of all UOR website and community media branding to Emporous.

[ ] Website repo
[ ] DNS / Domain records
[ ] Google Groups
[ ] Youtube
[ ] Twitter
[ ] Twitch

@usrbinkat Adding one. I saw we have a StackOverflow page.


redirect bug fix

Connection with issue(s)
reopen from PR
To Do

[x] double check the original issue to confirm it is fully satisfied
[x] add testing notes and screenshots in PR description to help guide reviewers
[x] request the "UX" team perform a design review (if/when applicable)

I forked your new project, but I can't make a merge request, I can only copy the old merge request, but it looks like there is a conflict, I don't know how to fix it, please let me know if you know

@lyb5834 merged. when you want to open up a PR, open from this repo. that was the issue.

@arrrrny Okay, I got it, it looks like you've merged this commit, thanks


Fix fragmentation check on uxr_prepare_reliable_buffer_to_write

Fix fragmentation size check
Integration tests running on https://github.com/eProsima/Micro-XRCE-DDS/pull/159

Build status:

Linux
Windows

Build status:

Linux
Windows


[Arm Unix build] arm variants of System.Native.a and libSystem.Globalization.Native.a are needed for cross building tests

For cross building corert tests for arm architectures (see PR #2682) arm variants of System.Native.a (and libSystem.Globalization.Native.a are needed. In CoreFX System.Native.a is built from the sources. May be the same way is applicable in CoreRT also?

CoreRT gets System.Native.a and System.Globalization.Native.a via nuget package. They get build in CoreCLR/CoreFX repos and published from there.

@jkotas Could you tell  which way can I get arm versions of this libraries on Linux x64 via nuget?

System.Globalization.Native.a is in Microsoft.NETCore.Native package
System.Native.a is in runtime.native.System package

I would expect that the arm builds for these packages may work same way as arm packages (e.g. https://github.com/dotnet/coreclr/pull/8421).
In the meantime, you should be able to just build them in corefx/coreclr and copy them over where you need them.
(As you can tell from the inconsistent package names, there is some work to do to get this cleaned up.)

CC: @Dmitri-Botcharnikov

@jkotas
yes, I did as you say, built these libraries in coreclr/corefx and copied them into corert. After this the c++version of Hello test is linked successfully.


fix: hopefully fix syntax highlighting and update headings

The following pull request (hopefully) fixes the problem where Rust codeblocks are not properly syntax highlighted in https://docs.top.gg and also update some headings.
Another pull request in the rust sdk has been created to reflect this change: https://github.com/Top-gg-Community/rust-sdk/pull/19

Stfu netlify
you always fail

Hey @velddev, can you merge this pull request for me?


PMID:30201262

NTR: actin patch aggregation during interphase
(is this a morphology phenotype?)

NTR: decreased rate of actin cortical patch localization | Microscopy |   | to cell division site

aggregated actin cortical patches during mitotic interphase FYPO:0007200
decreased rate of actin cortical patch localization to cell division site FYPO:0007201
I've put FYPO:0007200 under mislocalized patches, to match the existing patch aggregation term. As far as I know, the morphology of each patch is (or at least could be) normal; they're just much closer together than normal.
edit file: 1a509f11fdd6222a01744ca6f1e083623d2f8aeb
release: 0fc81ac9fd6927113bee9fed420f74b25bdde377