Cache line size option

Zic64b.   64-byte cache lines

Cache lines? Or cache blocks? The Zicbo* extensions work on "cache blocks," but the definition was crafted to allow us to abstract the implementation of the caches, i.e. the size of a cache line in the implementation doesn't have to match the size of a cache block. In other words, we were thinking "cache block" was a software concept that could map into one or more "cache lines" in the implementation of the cache. Of course, that mapping is trivial if the size of a cache block equals the size of a cache line.
There's also some subtlety with respect to coherence management and writeback data size (and maybe even reservation sets?). Does this extension imply all of these things are uniformly 64B?

@dkruckemyer-ventana It should be "blocks", since the intent was to specify the unit handled by Zicbo*.  (And indeed the existing normative text in the profile spec uses the word "blocks".)

The Zicbo* specification mentions two different block sizes in section 2.7: 1) cache block size for management and prefetch instructions, and 2) cache block size for zero instruction. Would Zic64b imply a 64-byte cache block size for both?

Should be Zic64b. 64-byte cache blocks.

@cmuellner Yes, Zic64b implies 64-byte lines for both 1) and 2).


add travis ci testing

and lint tests with shellcheck
fixes #84

when you get travis set up, drop me a comment here and I'll push another commit to kick off a test.

@msimerson Alrighty. I have signed up for Travis. If I had known it was a simple sign up form, I'd have done it right away. ;-)
---Alex

Okay @aqw , this PR is complete and you can see the results of running the lint tests against all your files.
I've disabled a handful of shellcheck tests but you ought to review them and make sure my guesses about how you'd choose were accurate. There remains a handful of shellcheck lint errors that I think should be changed in the scripts, but I'll leave that to you to decide if you prefer to fix or suppress the warning.
Once those are addressed, then the Travis CI tests will pass and you'll get automated tests run on every PR.

I have updated this PR further:

I updated the code with a couple POSIX updates (see commit messages)
I disabled a few more shellcheck tests (see tests/lint.sh & inline # shellcheck disable=...

Now the test suite is automatically run and the build passes. This PR is now safe to merge.
However, I'd still recommend going through the list of disabled tests. In a couple cases, I think it would be better to change the code than disable the test, but for now, I'm erring on the side of "make fewer changes."

ping

Hey @msimerson,
Thanks for the ping.
Quick mind-dump, as I'm in the middle of an all-consuming project at work that should be done in 4 weeks. After that, next on my list is zfsnap; cleaning things up, merging PRs, and finally, oh so finally, getting 2.0.0 released.
I've left a few comments on your commits, but my overall question is: can travis do what we need it to do? I have neither setup nor used travis before, so it's a question from ignorance rather than a counterargument.
More than anything, zfsnap needs to be tested across as many available shells on linux, FreeBSD, OS X, smartos, and Solaris. Different shells behave differently on different systems. The joys of portability...
The current vagrant system we're using is insecure and not easily used as CI.
 Then we can have whatever OS targets we want.
So, the question for me is: how flexible is travis when it comes to integrating with private CI boxes?
---Alex

Your unknown preference is why that particular change was in it's own commit. So you could, if desired, use cherry-pick and only merge the others. I don't know how portable it is, but IIRC, it's part of the POSIX shell spec and bash for a very long time. Since you asked, here's some references for that change:

Shell Command Language
BASH FAQ Why is $(...) preferred over ... (backticks)?
ShellCheck SC2006

@msimerson Ok. I'll be sure to drop that commit if/when merging.
I understand well why $() is superior to ``; I use it all the time. I can't remember which shell was problematic; it's in my notes somewhere; it was 3 years ago.
---Alex

I have removed the $() commit and added SC2006 to the exclude list.


updating freshness

for content sets with known values

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/aws-professional/index.md
:white_check_mark:Succeeded
View

docs/aws-professional/services.md
:white_check_mark:Succeeded
View

docs/checklist/availability.md
:white_check_mark:Succeeded
View

docs/checklist/resiliency.md
:white_check_mark:Succeeded
View

docs/checklist/scalability.md
:white_check_mark:Succeeded
View

docs/patterns/cache-aside.md
:white_check_mark:Succeeded
View

docs/patterns/category/availability.md
:white_check_mark:Succeeded
View

docs/patterns/category/data-management.md
:white_check_mark:Succeeded
View

docs/patterns/category/design-implementation.md
:white_check_mark:Succeeded
View

docs/patterns/category/management-monitoring.md
:white_check_mark:Succeeded
View

docs/patterns/category/messaging.md
:white_check_mark:Succeeded
View

docs/patterns/category/performance-scalability.md
:white_check_mark:Succeeded
View

docs/patterns/category/resiliency.md
:white_check_mark:Succeeded
View

docs/patterns/category/security.md
:white_check_mark:Succeeded
View

docs/patterns/circuit-breaker.md
:white_check_mark:Succeeded
View

docs/patterns/compensating-transaction.md
:white_check_mark:Succeeded
View

docs/patterns/competing-consumers.md
:white_check_mark:Succeeded
View

docs/patterns/compute-resource-consolidation.md
:white_check_mark:Succeeded
View

docs/patterns/cqrs.md
:white_check_mark:Succeeded
View

docs/patterns/event-sourcing.md
:white_check_mark:Succeeded
View

docs/patterns/external-configuration-store.md
:white_check_mark:Succeeded
View

docs/patterns/federated-identity.md
:white_check_mark:Succeeded
View

docs/patterns/gatekeeper.md
:white_check_mark:Succeeded
View

docs/patterns/health-endpoint-monitoring.md
:white_check_mark:Succeeded
View

docs/patterns/index-table.md
:white_check_mark:Succeeded
View

docs/patterns/leader-election.md
:white_check_mark:Succeeded
View

docs/patterns/materialized-view.md
:white_check_mark:Succeeded
View

docs/patterns/pipes-and-filters.md
:white_check_mark:Succeeded
View

docs/patterns/priority-queue.md
:white_check_mark:Succeeded
View

docs/patterns/queue-based-load-leveling.md
:white_check_mark:Succeeded
View

docs/patterns/retry.md
:white_check_mark:Succeeded
View

docs/patterns/runtime-reconfiguration.md
:white_check_mark:Succeeded
View

docs/patterns/scheduler-agent-supervisor.md
:white_check_mark:Succeeded
View

docs/patterns/sharding.md
:white_check_mark:Succeeded
View

docs/patterns/static-content-hosting.md
:white_check_mark:Succeeded
View

docs/patterns/throttling.md
:white_check_mark:Succeeded
View

docs/patterns/valet-key.md
:white_check_mark:Succeeded
View

docs/resiliency/failure-mode-analysis.md
:white_check_mark:Succeeded
View

docs/resiliency/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Compiler Option --outDir should remove existing files

TypeScript Version:  2.1.4 on Windows 10
Expected behavior:
When running tsc -w with outDir set, it should remove existing files.
Actual behavior:
Old files are left in the output directory and it seems that sometimes they block the creation of the new files.
I do have other build tools watching the output directory and I wonder if this is a problem, because if I try to delete the folder manually while the build tools are running, I get a warning that the directory is locked.

The compiler does not delete files. deleting files entails user data loss, and it is not usually a thing that compilers do.
My recommendation is to add a clean task that deletes your outDir all together.

Ok, thanks

The compiler does not delete files. deleting files entails user data loss, and it is not usually a thing that compilers do.
this comment doesn't really line up with a compiler that can be (and is recommend to be) run in watch mode.  In watch mode, the compiler is happy to overwrite files that it's generated, which entails the same amount of data loss.  Deleting those files when the source files that generated them go away seems like the consistent thing to do, and needing to clean out all compiled files and start fresh seems like it defeats the purpose of incremental compilation with --watch.

Would you be open to a PR supporting this behavior behind a flag?

overwriting is different from deleting. deleting will have to be a complete directory, since the compiler does not what it wrote to it last time it ran. and deleting a whole folder is a very drastic action, and I would not be comfortable with a tool that does that un-announced.


AutomationRemoteArray: Add equality comparison methods

Remote arrays can be compared for equality in the underlying Remote
Operations VM. This PR exposes this functionality via the builder methods.
Notably, this allows Runtime Ids to be compared more efficiently than before.
(Previously, it was possible, but the Remote Op itself needed to implement
element-by-element comparison, which would count against the instruction
limit, waste time at runtime, as well as was more complex than it really should
be for an important and relatively common thing.)

/azp run


style(Algebra/Order/Monoid/Lemmas): Update to current naming convention

Per the wiki, mul_le_cancellable should translate to MulLECancellable. Also changes mul_le_cancellable_one to mulLECancellable_one.

bors merge


error importing font-awesome and bootstrap

i get the following errors:
ERROR in ./~/css-loader!./~/stylus-loader!./client/app/app.styl
Module not found: Error: Cannot resolve 'file' or 'directory' ./node_modules/bootstrap-stylus/stylus/bootstrap.styl in /Users/dror/workspace/mng-ui/dashboard/client/app
@ ./~/css-loader!./~/stylus-loader!./client/app/app.styl 3:10-117

Please read stylus-loader documentation, they have example in "Usage" section.
Also note than bootstrap-stylus is deprecated, use bootstrap-styl instead.

i'm using bootstrap-styl
that resolves an error.
is there an example project with bootstrap imported?

@i-am-dror i want to add wiki page about this topic. Personally I use angular-ui-bootstrap with less in my projects.

i have no problem requiring the angular-ui-bootstrap module,
the problem is importing the styl, i get error regarding the inner dependencies.
all of the files are directed to "bootstrap/..." and its not resolving.

Did anyone figure out how to load bootstrap yet? It's proving to be very challenging.
The documentations aren't clear and the paths seem to be wrong

@PedroEsperanca Please see https://github.com/AngularClass/NG6-starter/pull/129 - there is an example.

thanks for the quick response @fesor !
My difficulty was with bootstrap-styl, but I ended up using bootstrap-sass since it has a larger usage

Hi, there is no mention as to how we can include font awesome??


feature request: shuffle for array with desired array length output

proposed feature request:
const arr = [1, 2, 3, 4, 5, 6, 7, 8];
const result = shuffleArray(arr, 3); // returns [3, 5, 8];

arr will be shuffled first
carret will pick 3 random items from the shuffled arr
returns the result

will this be possible? I'd like to use it on one of my projects. thanks!

Good idea. I will add this feature now.

Added in v1.0.1.


Refactor with hooks

This project was originally created with hooks. It doesn't have class components.

Hi @teitonny
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations!

H
Highlights

No linter errors :heavy_check_mark:
Repo looks professional :briefcase:
Solid pull request scroll
Good quality commits
Good job getting approved on your first try!! :clap: :clap: :clap:

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Add user account request

Currently, the only way for a user to request access to a role is via a mailto link to the vector atlas email address. This story involves adding a section to the user settings page created in #281 which allows users to select roles they would like to be added to their account, give reasoning, and submit. This then fires an API action which emails an email list (from config) with the request and reasoning, suitably formatted. This section should be controlled by a feature flag.
This new page should be linked to from the Unauthorized page in the AuthWrapper, instead of the mailto link.
Adding email functionality is explained in https://github.com/icipe-official/vectoratlas-software-code/issues/165. When this is added to the nestjs project, include an environment variable flag in the app.module file (e.g. disableEmail) which ensures the API builds without this email module.
AC's:

[x] The user settings page has a new section, allowing a user to select roles they would like assigned, and fill out a free-text field for reasoning. There is also a 'Submit' button.
[x] Clicking on the 'Submit' button causes an email to be sent to the email address(es) configured in the API config, with the 'ADMIN_EMAIL' env variable
[x] This new page is linked from the Unauthorized page.

Moving back to "In Progress" - as discussed, the "Unauthorized page" isn't linking to the user settings page.

We also discussed changing the state of the "Submit Request" button once a request has been sent to include a message saying that the request has been sent successfully and to force a  user to refresh the page before any further requests can be sent - to help prevent accidental/spam requests.

New role request form on the user setting page:

Example role request email:

User setting page now linked from "Unauthorised" page:

The user role form now resets once a request is sent and the success message appears for much longer making is easier for a user to see it

The form can be submitted with no input (i.e. blank), a bug has been created: https://github.com/icipe-official/vectoratlas-software-code/issues/395

Bug #395 is closed.

Tests 7.8 and 7.9 added to test script.


Getting the job workflow working

So that once a job has been accepted, only the seller can start it, only once a seller has started a job can it be completed, only once a seller has completed a job can the buyer pay for the job

Interesting problem: in Firestore, how do you update a child element.
Say you have a Scoop, and the Scoop has an array of Jobs. You click on the detail page for a job and update it (e.g., set its AcceptedAt date to the current timestamp). How do you update a property of an element in an array?

I am going to try something like:
val jobRef = db.collection("scoops").document(scoop?.scoopId!!)
jobRef.update("jobs", FieldValue.arrayRemove(job))
job.acceptedAt = Timestamp.now()
jobRef.update("jobs", FieldValue.arrayUnion(job))
Seems super hacky, but I do not know how to update a particular job within the jobs array. Any ideas appreciated.

Ok - that works - but it does not seem very elegant. If you know how to update

an attribute
of an element
in an array
in Firestore
with Kotlin
please check out the code and tell me how to do it. Here is how it looks

else if (job.paidAt==null)
    {
      itemPay.isEnabled = true
      itemPay.setBackgroundColor(Color.CYAN)
      //itemAccept.visibility = View.VISIBLE
      itemPay.setOnClickListener {
        db= FirebaseFirestore.getInstance()
        val jobRef = db.collection("scoops").document(scoop?.scoopId!!)
        jobRef.update("jobs", FieldValue.arrayRemove(job))
        job.paidAt = Timestamp.now()
        jobRef.update("jobs", FieldValue.arrayUnion(job))
        handleWorkflow(job, scoop)
      }

And here is a link to the code: https://github.com/dyor/scoop/blob/main/app/src/main/java/com/dyor/scoop/JobDetail.kt

Here is an updated UI - as you click on each step the app sets the timestamp for that step and advances to the next one.

Next step: make it so that the business logic only lets the proper users click buttons (e.g., any Seller can accept a job, only the Seller who accepted the job can start and complete the job, and only the Buyer can pay for the job).


Export CasePathsMacros

Hi @ImSingee, can you explain a bit why you need access to the CasePathMacros module? Are you building a tool that uses the various macro conformances somehow? And if so, can you share more details?
We would be happy to publicize this module if it's useful. We just aren't aware of any actual uses for it.

@mbrandonw Ah I just think it's necessary.
I can't use @CasePathable macros... It always show an error External macro implementation type 'CasePathsMacros.CasePathableMacro' could not be found for macro 'CasePathable()'

That should work fine. There's no need to publicize the macros module. Can you share a minimal reproducing project that demonstrates the behavior you are seeing?

@mbrandonw So strange😂 I just remove my xcuserdata and then it works.
Sorry for wasting your time...


Feature(ConnectorSdk): PriorityFirstSync Example

Links https://fivetran.height.app/T-767058
PFS Example
Testing
Tested locally. Both incremental and historical syncs work fine.

Have made the changes. It's ready for review.

@varundhall Addressed most comments. I've replied on 2. Pls check.

@fivetran-satvikpatil please check locally once before approving the PR.

please address the minor comments before merging,
also I think you need a init.py file, as its a package.

I think I will be skipping. From what I read online, it's not required anymore. It maybe was required for older python versions.

Please make sure you example works properly by running fivetran debug at least few times.

Sure
Cc: @varundhall

@varundhall pls release the python library with Rajat's fix. Else it will not work for Satvik.

When we run fivetran debug, we don't install the library using requirements.txt. So it was throwing me the error regarding this library. I installed it manually, it is working fine.


Add GitHub actions building for dockerfiles

(title of pull request)

fixes #33

Reviewers: @taliamax
Description:

Fix up the frontend prof and backend dockerfiles and make GitHub actions build them

It appears to already be referenced as an env_file in the docker-compose.yml which is why I decoupled it.
An env file with secrets should not be committed in any way, we should use runtime environment variables in the docker container when run.

Build steps renamed more clearly. Bonus, backend and frontend split into steps for parallel execution.


Add startup for client/mono Blazor

This change adds a host builder, and the startup pattern for client-side
Blazor apps running in mono/wasm. This will help us align better with
server side Blazor.

Note that I haven't done anything to the template yet. Waiting for feedback and then I'll add that on.
My next step after this one is to start building out the server project.


bluebird promisify not working with node-bcrypt compare (does with hash)

I am trying to rewrite some callback hell auth to use promises (which I am relatively new to).
import bcrypt, {hash as bcryptHash, compare as bcryptCompare}  from 'bcrypt';
Promise.promisify(bcryptHash);
Promise.promisify(bcryptCompare);
...
  hash(unhashed) {
    const genSaltRounds = 10;
    return bcryptHash(unhashed, genSaltRounds);
  }

  verifyHash(unhashed, hashed) {
    console.log('authServiceUtil bcrypt.compareSync', bcrypt.compareSync(unhashed, hashed));
    return bcryptCompare(unhashed, hashed);
  }

Calling the hash function above works as expected, but not verifyHash, which is always returning undefined. The console.log with the compareSync does work (and prints true in this case).
Any ideas why this does not work?

@rkoberg .promisify returns a new function, but you're throwing the result of the call away.

Oh geez, sorry. I had been using promisifyAll and thought they worked the same. (read the api docs about promisifyAll, but not promisify)


"Too many arguments for format string" warning on every failure with $VERBOSE = true

I recently updated from Ruby 2.7 to Ruby 3.1; as a part of this, I set $VERBOSE = true in the development environment.
I now get a warning with every error thrown by database_consistency:
/usr/local/bundle/gems/database_consistency-1.3.9/lib/database_consistency/writers/simple/base.rb:50: warning: too many arguments for format string
ColumnPresenceChecker fail User password column (password) is missing in table (users) but used for presence validation
/usr/local/bundle/gems/database_consistency-1.3.9/lib/database_consistency/writers/simple/base.rb:50: warning: too many arguments for format string
ColumnPresenceChecker fail Event calendar_feed association foreign key column should be required in the database

Unfortunately, that's in a fairly generic method, and warnings don't generate full stack traces, so I can't work out where the format string is coming from 😞

Hi @unikitty37,
Thank you for letting me know! I will improve it as soon as possible.

Hey, I've just released 1.4.1. Could you please try and let me know if that helped?
Feel free to reopen an issue if needed.

Thanks — that's fixed it! 👍


Add views to publishing

I'd like to be able to customise the views, but they're not tagged for export.

Just copy the views to your Spark application under resources/views/vendor/google2fa/.


Mount kmc secrets instead of using the env variables

Description
Changes proposed in this pull request:

Read the EDP token from the pod filesystem
Remove the EDP token from env vars inside the kmc deployment

Related issue(s)
https://github.tools.sap/kyma/backlog/issues/2010

/test required

/retest

/test all

/test pre-main-control-plane-reconciler-upgrade-kyma2-latest-to-main-gardener


Incomplete translations of Chinese documentation

Issue Description
There are two main issues with the non-English documentation for OpenTelemetry:

Edit (@chalin): this will be tracked via #4898
~Broken links in Chinese, Japanese, and Portuguese documentation:~

~In these language versions, clicking on "API" or "Examples" under "Language APIs & SDKs/Python" leads to a "not found" page; for example, under chinese page it if you click API, it will jump to https://opentelemetry.io/zh/docs/languages/python/api/~
The correct links should be:

API: https://opentelemetry-python.readthedocs.io/en/stable/api/index.html
Examples: https://opentelemetry-python.readthedocs.io/en/stable/examples/

Incomplete translations:

The Chinese documentation appears to have limited translations, with much of the content still in English.

Proposed Solutions

Fix the broken links in the Chinese, Japanese, and Portuguese documentation to point to the correct API and Examples pages. I can try to do this.

Consider expanding translation efforts for non-English documentation, particularly for Chinese. As a community member, I'm willing to contribute translations for Python and Go documentation if there's a need.

Questions for Consideration

Is there significant demand for fully translated documentation in languages like Chinese?
Are there metrics available on the views/usage of non-English documentation pages?
Should translation efforts be prioritized, or deferred based on current demand and resources?

I'm open to contributing to this effort if the community feels it would be valuable.

Thanks for reporting issue (1) @zhihali. I'll be addressing it after the following PR is merged:

#4895

As for (2), that holds true of all non English localizations and is understood to be true :). Your contributions are welcome. Submit a PR. You asked good questions.  This would be fun 🙂

FYI, I repurposed this issue to focus only on (2). We'll track (1) via #4898.
We'll probably close this issue soon since, as I mentioned, we know that all the localizations are work in progress.

 Could you clarify if these view counts are from the past month, past year, or some other time period?

Hi @zhihali - I can't recall the default parameters used when I generated the report, but IMHO, the important thing here (relative to your question of which pages get the most traffic and would be most valuable to translate) is the order of the paths in that list.


Adapt HPA upscale/downscale delays

What this PR does / why we need it: We are using the default delays for the HPA (5m for downscale, 3m for upscale). We have seen that this might result in an oscillating behaviour in some cases (depending on the load, of course). With this PR we can hopefully reduce this oscillation.
Release note:
The delays for the horizontal pod autoscaler have been modified in the following way to achieve a less oscillating behaviour: Downscale: `5m` -> `15m` and Upscale: `3m` -> `1m`.

Maybe we can try to test it, to get more accurate values?

Can you elaborate? Which kind of test do you envision and how do you want to derive values out of it?

I just wanted to replace "we can hopefully reduce this oscillation" with "we will reduce this oscillation." Instead of trying one value and seeing what happens we can try to do  a load-test and adjust the values for the delays and see how the HPA will behave in different load scenarios with the configured values. This way we can be more sure of the behavior.

OK, but I'm not sure how a load test can help. I mean, the values are pretty clear described: When the HPA controller decides that it must scale up it now waits for one more minute (instead of three) until it really scales up, and when it decides that it must scale down it now waits for 15 minutes (instead of five) until it really scales down.
Unfortunately, these delays cannot be set per HPA but only globally as flags in the KCM. So, I'm not sure whether we will get anything out of a "load test".


#457 Mark Activities Complete/Incomplete in Calendar's ProgramActivityDialog

Proposed changes
This PR resolves #457.
Minimum Requirements:
This PR adds a 'Mark Complete/Incomplete' button to the ProgramActivityDialog when the activity type is an assignment. Users can mark activities as completed or incomplete by clicking the button. The frontend sends a GET request to the backend to retrieve the activity status when the ProgramActivityDialog is open and sends a PUT request with the updated status to the backend when the button is clicked.
Stretch:
The Task Alt Icon shows on the right of the activity title when the activity status is completed. This PR also adds the isLoading state for both GET and PUT requests, so the spinner shows in the button when the API request is loading. If the PUT request is successful, the success message will show in the toast/alert on the bottom left. Error handling is also included to show an error message in the toast/alert when errors occurred for both GET and PUT requests. When the GET request is not successful, the button will be disabled to avoid the user sending a PUT request.
Not required for this ticket but the close button on the bottom right is replaced by the small close icon on the top right.
Checklist

[x] Are the issues being addressed linked to this PR?
[x] Do all commit messages start with the issue number?
[x] Are all code changes sufficiently tested?
[x] Are there screenshots for UI changes?

@Prabhj33t I can't reproduce that screenshot that you have posted since your database is out of date; that activity doesn't exist anymore. Can you run the following commands and see if you can reproduce that behavior?
cd ~/Developer/rhizone-lms (or wherever you keep the repo)
git checkout 457-mark-task-completed-in-calendar
git pull
docker compose down && docker compose build --no-cache --pull && docker compose pull && docker compose up

Also,

The x button overlaps with the check sign here otherwise, all seems good. I approve of this ticket

For pull request approvals, make sure to mark your comments as an "approval"; it defaults to leave the review as a "comment" status, which won't count for one of @wendyyng's two approvals needed, unfortunately.

Great work! All the tasks as expected.
_'Mark Complete/Incomplete' button works
_ mark activities as completed or incomplete by clicking the button works
_If I change the code to fetch(abc.com`` in ProgramActivityDialog in line 41 => it shows disable button with error "getting the assignment status"
_If change the code to "204" in programsRouter in line 102 => it shows error "updating the assignment status"

Update 12/14:

add space between defined variables and useEffect;
add empty line above sendAPIGetRequest in ProgramActivityDialog;
reorganize imports of icons from MUI in ProgramActivityDialog;
import useState, useEffect, Dispatch, setStateAction from React in ProgramPage, ProgramActivityDialog and clean up code;
specify the type of body of sendAPIPutRequest;
remove async from sendAPIGetRequest in ProgramActivityDialog;
import MouseEvent from React
working on error handling

Based on valuable feedback provided by @wendyyng this morning in standup, I've modified the error message display in the toast. In many cases, it will now be much shorter, looking a little something like this:

(Contrived example, obviously; the server wouldn't send a SyntaxError. But you get the point.)
Let me know if that adequately resolves the feedback, and if it does, let's merge this PR!


Move file to www folder

BEGINRELEASENOTES

Move test files in key4hep eos folder

ENDRELEASENOTES
Fix #10

Great, Thanks. I may do some git history rewriting to remove the blobs.


Docker install

Hello,
Have you tried to install this plugin in a Docker environment? I have tried several ways but I cannot seem to make it work.
FROM kibana:5.5
MAINTAINER esteban.sastre<EMAIL_ADDRESS>
ENV KIBANA_PLUGINS /usr/share/kibana/plugins
RUN apt-get update && apt-get install -y git npm

WORKDIR $KIBANA_PLUGINS
RUN git clone https://github.com/dlumbrer/kbn_network.git network_vis && cd network_vis && rm -rf images/ && sed -i "s/5\..\.0/5\.5\.0/g" package.json && npm install

Last line just changes the package.json form 5.4.0 to 5.5.0 as you advised, but I cannot seem to find the plugin in visualizations anywhere. I have tried with other plugins and they seem to be properly installed.
Is is possible that you provide a way like: kibana-plugin install <url>?
When it comes to build docker images with plugins it is much easier.

If you are using Kibana 5.5.0 you need to install my plugin from this release:
https://github.com/dlumbrer/kbn_network/releases/tag/5.5.X


[W6.1d][T10-B3]Wang Jie

Good job! Almost there. I like how you think about the effects of adding isMutating and updating the tests and save


How to use sortable/meteor/example demo in my localhost ?

Hi, im newbie in meteor,
i already download it, this example of meteor/example. but i confused how to play it, there is no .html things like in home (sortable only, index.html). how i can play it in my local, like http://rubaxa-sortable.meteor.com/
thanks for any help.
sorry im newbie.

It's explained RIGHT in the readme! https://github.com/RubaXa/Sortable/tree/master/meteor/example

sorry, i have a problem with this terminal things (i dont know how to fix it), they said :
Warning: Permanently added the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
Permission denied (publickey).
fatal: Could not read from remote repository.
could you give me more detail explanation? thanks in advanced

How are you cloning the repo? Probably via SSH.
Try cloning via HTTP:
https://github.com/RubaXa/Sortable.git


Single Image and Canvas

Hi @auduno. This is more a question than an issue.
Do you have to use a Canvas with an Image rendered into it, or can this just be run on an run-of-the-mill Image tag?

Yes, you have to use a canvas element, as it is not possible to do common javascript image operations (reading/inserting pixels, rotating image, etc) with IMG elements.


Change start_prank so that it can be used with deploy_contract

Problem

doing start_prank(caller_address=123, target_contract_address=None) will only make the prank in the context of the test runner contract, not the contract actually being tested
doing start_prank(caller_address=123, target_contract_address=ids.deployed_contract) wouldn't really work as we would need the contract address in advance

Possible solutions

start_prank should affect all contracts by default
start_prank(caller_address=123, target="ANY")

start_prank should affect all contracts by default I we should do this, handling ANY is too complicated

start_prank should affect all contracts by default

+1. this feels like a sane default to me
Also, i don't think it matters in this case, but maybe worth considering the implications on delegate/library calls. IIUC it doesn't matter since:

... get_caller_address() and get_contract_address() will return the same value they would have returned if they were called from the calling function.

Seems it should be fine, but figured I'd mention it in case I'm thinking about it wrong :)

Closing, will be solved by https://github.com/software-mansion/protostar/issues/352


fix: smuggle hooks can return interface, as the doc always said

coverage: 99.866%. remained the same
when pulling f56408714c1f010b977def5d05e402f957eab46d on fix-AddSmuggleHooks
into 679b2d7f67c061f90714cf4fc0e7c4f5141c31c3 on master.


Prefer to use FETCHCONTENT_BASE_DIR when specifying include directories from FetchContent (Dawn)

This variable controls the point under which all content population directories are collected, also when not changed it points to it. If it was changed target_include_directories would now include nonexistent directory.
Now that I looked at it, ${dawn_SOURCE_DIR} was also defined. Either of them would work

This one

Good catch, thanks! Fixed by 35bad0c


Add new CNI focussed kubelet tests

Signed-off-by: Sai Sindhur Malleni<EMAIL_ADDRESS>Description

kubelet-density-cni: This creates deployments of node.js
webservers fronted by services and another deployment of pods that
run curl against the webserver service for readiness probe. This ensures
that the curl pods only go into ready when they are able to reach the
webserver srvice. kube-burner accordingly accounts for this delay in the
podready metrics.

kubelet-density-cni-policy: This is similar to above workload
except  a deny-all policy is created in the namespace to begin with and
later a newotk policy to allow http on 8080 is introduced when the
pods/services are created. This also ensures that the curl pods'
readiness probe only succeeds when the flows dictated by the
network policy are applied. kube-burner podready metrics now account for
propagation of this network policy.

I think it needs some rebase, to use the new centos stream repo:
STEP 6/11: RUN dnf install -y --nodocs redis openssl --enablerepo=centos8-appstream-* && dnf clean all
Updating Subscription Management repositories.
Unable to read consumer identity

This system is not registered to Red Hat Subscription Management. You can use subscription-manager to register.

CentOS-8-Appstream-x86_64                        21 kB/s | 8.1 kB     00:00
Errors during downloading metadata for repository 'centos8-appstream-x86_64':
  - Status code: 404 for http://mirror.centos.org/centos/8/AppStream/x86_64/os/repodata/repomd.xml (IP: <IP_ADDRESS>)
Error: Failed to download metadata for repo 'centos8-appstream-x86_64': Cannot download repomd.xml: Cannot download repodata/repomd.xml: All mirrors were tried
Error: error building at STEP "RUN dnf install -y --nodocs redis openssl --enablerepo=centos8-appstream-* && dnf clean all": error while running runtime: exit status 1
make: *** [Makefile:92: image-build] Error 125
Error: Process completed with exit code 2.


fix(iOS): sync cookies obtained in WKWebView to sharedHTTPCookieStorage

Summary
In our app the authentication is completed inside a WebView. All other interactions with the backend is using the native fetch implementation. When using UIWebView, all cookies obtained in the WebView was stored in sharedHTTPCookieStorage, making them available to the native fetch implementation. After upgrading to react-native-webview >= 7.x fetch could no longer find the cookies obtained in the WebView when using simulator. For some reason it still works on device.
The changes in this PR copies all cookies from WKWebView to sharedHTTPCookieStorage when sharedCookiesEnabled property is set on the WebView component.
The following comments in #312 describes the same issue, and should be resolved with this PR.
https://github.com/react-native-community/react-native-webview/issues/312#issuecomment-498774707
https://github.com/react-native-community/react-native-webview/issues/312#issuecomment-551300898
https://github.com/react-native-community/react-native-webview/issues/312#issuecomment-555489505
Test Plan
What's required for testing (prerequisites)?

iOS simulator
A website that sets a cookie

What are the steps to reproduce (after prerequisites)?

Visit website that sets a cookie using the WebView
Inspect sharedHTTPCookieStorage

Compatibility

OS
Implemented

iOS
✅

Android
N/A

Checklist

[X] I have tested this on a device and a simulator

@forsen I feel like this should maybe be behind a prop option. What do you think?

@forsen So this just fixed the simulator then?

At least in my use case, it does.
During onboarding the user must obtain a cookie through a webview that must be used on all calls to our API (using fetch).
This works fine on device, but stopped working on simulator with the transition from UIWebView to WKWebView.
With the fix in this PR it works fine both on device and on simulator in our case.

Can you resolve conflicts, @forsen?

Sorry about the delay, being busy having summer vacation. All conflicts are resolved.

Any updates on this potentially getting merged?

@nspector Probably in the next round of updates we publish -- within a month.

@nspector Probably in the next round of updates we publish -- within a month.

Any updates on when this will be merged?

@nspector Probably in the next round of updates we publish -- within a month.

Any updates on when this will be merged?

Hello 👋, this PR has been opened for more than 2 months with no activity on it. If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing! You have 7 days until this gets closed automatically

ping

ping


remove group status from day 1 and 2

remove group status from hackmd

Thanks @blankdots , I think this looks fine. I will merge this PR.


Fix Prometheus sample route definitions + typos

I will backport this change onto release/latest once #1596 is finished.

Oopse.


Update Composer dependencies (2024-09-04-00-18)

Visual regression test passed!
View the visual regression test report


References to "database" in tutorial

RE: https://github.com/openjournals/joss-reviews/issues/3617
There are references to a "database" in the tutorial - I don't know what these mean.  What database?  Are you talking about linking to external legal databases of some type?  Are these just adding URLs?
It also says things like "fetched from our database" -- but what is "our database"?

Related to the above, the entire section

Now we want to resolve all of the extracted cites into clusters indexed by the resource they refer to, such as a case or statute. We'll use a custom function to resolve a given full cite to its resource, so we can return our own MyCaseModel for citations we recognize. We'll fall back on returning resolve_full_citation() for citations we don't recognize.
For this simplified example, we'll assume we have a database model MyCaseModel so that MyCaseModel.get_by_citation() will return the case referred to by that citation string. In real life this might be a Django model or Elasticsearch lookup. We'll also assume that the same case has the parallel citations "1 U.S. 2" and "1 S. Ct. 2", so MyCaseModel.get_by_citation("1 U.S. 2") returns the same case as MyCaseModel.get_by_citation("1 S. Ct. 2").

I got lost.  I think you're saying you have a hypothetical database with a table model class in it, separate from this package.  This is super confusing for folks who aren't familiar with this type of setup or don't have their own database.  Do you have to use a database for the citation resolution?  All code in the tutorial (with maybe a few exceptions for special use cases, etc.) should be able to run by a potential user.  I can't test the resolve_citations() functionality because the example code uses a database I don't have (and there aren't instructions to set one up.

Hopefully this should be clear now too. Could you take a look and hopefully it'll be better? Thank you for the helpful feedback.

It's clearer, but I still think the code in the tutorial should be mostly run-able if possible.  For the custom_resolution_callback function, could you make a very simple one that just looked up values in a dict or something similar?  And also code to make your SomeCustomObject class?  This "class" gets used later for annotation examples too.
Or can you call one of the APIs if they're open (no key) to show a real example?
I'm not going to hold up the paper publication over this, but sorting out some of these things should help eyecite be more clearly and functionally a stand-alone package, that doesn't rely on such close integration with the systems it was developed for.

Awesome!  That seems really helpful.

I'm going to go ahead and close this one. Seems like it's all set, but please let me know if I'm too eager.


Tree tool tips should display something besides Thing ID

Currently, if a user places their mouse over a tree, a tool tip will appear that displays the Thing ID for that tree. This is a pretty cryptic and unuseful piece of information to provide the user.
Instead of displaying the Thing ID in the tool tip, one of the following could be displayed:

the species of the tree
the adoption status
nothing (no tool tip at all)

FOR REFERENCE:
I believe that this is controlled in app/assets/javascripts/main.js.erb -> function addMarkersAround -> success: function -> thingTitle (var thingTitle = thingType + '_' + thing.id;).

I pushed a solution (eliminating the tool tip) to the 2017_05_23 branch, which I have submitted in a pull request.


.exe loading bar

Hey,
First off, this is excellent! By far the most stable and straightforward solution for generating (in my situation, offline) apps from html. Great work :)
I've got a question though, is there a way to create some kind of loading/progress bar for the generated .exe? The app is nudging 400mb and can take a while to load on older machines with no indication that anything is happening. While this is mainly down to hardware issues my aim is just to try and improve the users' experience when they first get hold of the app, if possible.
Have you got any suggestions?
Cheers :)

Hey! Thanks :)
One thing I'm working on for the next version is a different packaging option. For big files, it is more beneficial to package the app with a separate folder for all of the files that you will use that are large.
However, for right now, the only thing I can recommend is using javascript to load the large files on the fly, so they aren't loaded into memory straight off the bat. For example,
var dirPath = path.dirname( process.execPath );
var filesPath = dirPath + "/mycoolfiles/";

var fs = require('fs');
fs.readFile(filesPath + '/giantfile.mpeg', function (err, data) {
  if (err) {
    throw err;
  }
  console.log('Do stuff with my big file.');
});

Remember also that if you load files like this, you'll have to make sure they are not in the project folder, or else they will still be included in Web2Exe's packaging. I'm also working on excluding folders from the build.
Cheers!

Yeah, the javascript isn't really going to work because there's about 120 PDFs and 30 odd MP4s!
But your saying I could package the app to include only an index page and THEN move all the required files to the project folder? That would (a) keep the exe file down to a few kb and (b) make updating a whole lot quicker. Will have a play :)

1 excellent

I read javascript , css and resources will be packaged into one, this is hugely beneficial, it saves me from having to spend 1-2 minutes combining each file into a single.
then a bonus to this would be minification .

I have been watching this...
I say progress bar belongs in the installer. Or if extracting to a temp directory, the progress bar belongs with the program extracting.
Just my thoughts...

Perhaps a progress bar wasn't necessarily the correct request. More of a splash screen, or even just an image, just to give the machine a little bit of time to sort itself out in the background. Having done a bit of research on all this though I fully understand the headache of trying to implement this sort of feature.
I've seen chromium args which mention enabling/disabling boot logos/animations (even a chrome logo would suffice in my case) but I don't know if they would even work :/

Yes, I agree that some sort of loading bar is necessary for being packed into a single exe, but to my knowledge, I have no idea how to implement such a feature.
My solution to this is going to be different modes of packing the exe, one of which allows for external folders and files that will be loaded as needed instead of at runtime.

Maybe I am alone, but personally I package my completed software with a installer. Even if it is a "portable software" package I do the same.
Have always felt the "progress events" should be left up to installers. I use http://www.installpackbuilder.com/ and it will do everything mentioned here.
As longs as your software has option to turn off progress/splash image guess it really makes no difference though:)

Thanks @2braincells2go I'll give that installer a go, but once it's installed would it actually load any faster?
I played with app.js which obviously uses a content/ folder to pull up the assets. While this keeps the exe to a very small size, and therefore loading times, it has a few missing features such as triggering a html video to play full screen. W2E is by far the most complete solution I've found but it's just the loading time that's a stumbling block.
The main reason I'm using a packager such as this anyway is because the project loads pages with ajax. This works fine if you just load up the local index page in firefox but chrome freaks out with cross domain issues -_-

I use paquet builder along with the built-in UPX compressor (info in help docs about setting up). Helps speed things up some and certainly makes the package smaller.
Something I have learned over the years: If a user is staring at their screen with nothing "visual" happening, it seems twice as long to them. Give them a splash screen with some movement and complaints darn near disappear.
Not sure if we should continue this conversation here or not. @jyapayne  may be tiring of us, lol.
Cheers!

Yes, please move the conversation to email or whatever you like.
I will attempt to make changes that will alleviate the loading, since @2braincells2go is right about the user experience.

I'm also working on excluding folders from the build.

+1 to this!!!

:) This is quite a difficult change, and I'm still working out how to do it properly. There needs to be two modes of exclusion. One mode for just plain excluding folders completely, and the other for excluding folders that you then would like to access from inside your app.
The first one is not that difficult, but the second one requires creating a javascript file with some path modifications to let the app access the files like normal. I'll see what I can come up with.

Hello Sir Joey!
"One mode for just plain excluding folders completely, and the other for excluding folders that you then would like to access from inside your app."
Did you ever find a way load external files from folders within the compiled app directory?

Hey Woody!
Unfortunately, not really. You can load external files in Javascript if you check the "Uncompressed Folder" option  in the "Compression Settings" tab with NW.js versions greater than or equal to 0.13.0, but I could never really figure out how to set this up to work nicely by just having an option in the interface. If you need more detail, I could probably whip up an example.

Ok, totally understand if you can't work the magic on the interface.
Experimented using javascript but it failed. Might be because used compression and older NW.js, can't remember. Will try it again.
When you have time, example would be nice but no big hurry.
Thanks Joey!

Hey Joey. Hope you are doing good.
I have been playing around with external files and not having much luck. Back in April you had said:
If you need more detail, I could probably whip up an example.
Was wondering if you could provide some real life examples when you get a chance.
Thanks!
Woody

Hey Woody,
I've created a sample project that loads an image from outside the project directory. The project is source is located in the package.nw directory. You can run the External File Test.exe to see it in action. Hope it helps!

Your a champ Joey. I thank you much.

Sorry to keep bothering you Joey but cannot seem to find the files?
The project is source is located in the package.nw directory.

Haha, oops! I forgot to link the file! Here it is

Thanks Joey. I have been trying to include HTML files without any luck. Cannot seem to find any clear documentation on how to do this with javascript. Is it not possible?
Cheers!

It depends on what you mean by include. Do you want to display the HTML on a new page or within the page you are on?

It depends on what you mean by include. Do you want to display the HTML on a new page or within the page you are on?
What I was attempting to do is both actually. Have a software that has some mock math tests. Wanted to display a small banner  at bottom to alert when test has been changed/updated. Wanted the HTML to be totally separate. Rather not use iframes as have been doing in past.
Plus, was trying to learn little more about including external files because few students are asking me... I have not a clue, lol. There is sooooo much (mis)information and stackoverflow has totally confused me:)

Hey Woody,
Here's another example using an html file instead of an image. It's basically the same process. The only way I found to load a new page externally right now is to load the contents of the file and display it in a new window.

Thank you again Joey for taking time to help, it is much appreciated.
This is further than I have gotten and been working on it off/on for months, lol.
Are there any special plugins I need to use when making my own? Wondering if I am missing needed plugins as did a similar test month or so ago and failed...
Cheers brother!

Nope, all you need is in the package.nw folder and the external hello.html. If you export that using Web2Exe, and then copy the hello.html file to the folder that has windows-x64 in it, you should get the exact same result.

Good deal Joey. Most likely I fat fingered something. Tried so many different ways using NW forum and Stackexchange, have lost count. Amazing how much info is out there and much of it doesn't work even with their example files. I am going to have to find time to take a course...


Error message when loading segmentation data

When reviewing some tracer segmentations, I receive this error message (below). I have tried in both v0.3.1 and v0.2.18, and encountered the same issue. I notice that one of the following three scenarios happens: 1) it works as expected, 2) it opens the segmentation data but expected 'done' information is not retained, and 3) error message and cannot open the segmentation file. I will send the files I used when encountering the error message:

Believe the issue was incorrect renaming of a .nii.json file as a .nii file. Closing for now.


🧐 Coverage from instrumented tests

Adds custom jacoco task to merge coverage from instrumented and unit tests

Codecov Report

Merging #56 into master will increase coverage by 42.43%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master      #56       +/-   ##
===========================================
+ Coverage     1.9%   44.33%   +42.43%
===========================================
  Files          64       64
  Lines        2101     2102        +1
  Branches      167      167
===========================================
+ Hits           40      932      +892
+ Misses       2060     1120      -940
- Partials        1       50       +49

Impacted Files
Coverage Δ

...pro/surveyor/activity/BaseSubmissionsActivity.java
3.44% <0%> (+3.44%)
:arrow_up:

...in/java/io/rapidpro/surveyor/utils/ImageUtils.java
4% <0%> (+4%)
:arrow_up:

.../src/main/java/io/rapidpro/surveyor/data/Flow.java
96.55% <0%> (+6.89%)
:arrow_up:

...ava/io/rapidpro/surveyor/activity/RunActivity.java
21.1% <0%> (+21.1%)
:arrow_up:

...va/io/rapidpro/surveyor/widget/ChatBubbleView.java
23.4% <0%> (+23.4%)
:arrow_up:

...ain/java/io/rapidpro/surveyor/utils/JsonUtils.java
87.5% <0%> (+25%)
:arrow_up:

.../rapidpro/surveyor/net/responses/TokenResults.java
33.33% <0%> (+33.33%)
:arrow_up:

...va/io/rapidpro/surveyor/ui/CachedLinearLayout.java
33.33% <0%> (+33.33%)
:arrow_up:

...va/io/rapidpro/surveyor/activity/BaseActivity.java
46.59% <0%> (+46.59%)
:arrow_up:

...n/java/io/rapidpro/surveyor/SurveyorException.java
50% <0%> (+50%)
:arrow_up:

... and 36 more

Continue to review full report at Codecov.

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


Configurable tolerance

Thanks a lot for this library! I really appreciate how flexible it can be used for different use cases.
I was wondering if it could be possible to allow the consumer of the component to configure the level of tolerance?
E.g. in your menu example: When hovering over the menu, the items change their state very responsively when moving the mouse straight up and down or of course slightly to the left. However, if the mouse is moved slightly to the right the menu isn't so responsive anymore. Of course, there needs to be a boundary somewhere, but it would be nice if this was configurable.
Thanks in advance!

@amannn I agree that it would be nice if it could be configured. If you have time, feel free to submit a PR.

We need this too for our project, so we may look into adding some kind of minimum velocity setting.

@cwick please feel free to submit a PR.


PositionComponent should maintain its location when transferring to another parent

When component A is transferred from parent P1 to parent P2, we should preserve its global coordinates so that this transfer of ownership wouldn't cause the component A to "jump".
For example, suppose A is an arrow, and P1 is the world. As the arrow makes collision with an unfortunate NPC P2, we want to transfer the ownership of A to P2 so that visually it would become stuck in NPC's forehead. However, we want the arrow to remain physically in place during the transfer so that it sticks exactly at the point where it hit the enemy.
Another example: in a card game you pick up a card from the deck in order to drag it somewhere else on the field. When the card is picked up, it must be moved to a special transparent top-most layer, so that when dragging it wouldn't accidentally go underneath other cards on the field. Then, when released, it would have to be transferred into the component over which it was released. All these transfers of ownership should be completely invisible to the player, and visually the card should not change its position during the transfer.

I think this would make sense as an optional option, but not the default. Maybe a flag to the changeParent method?
In many 2D games things do jump and if the user has set a local position for something it should be maintained, imagine something like swapping a weapon from one character to another, then the weapon shouldn't still be on top of the first character.

I imagine you could kill an NPC, who drops a sword on the ground, then you pick it up and it goes either in your inventory or maybe directly into your hand -- but in any case you'd have a logic for where to place the sword on your character's sprite. So that logic would adjust the sword's position after the mounting, and it wouldn't care what the sword's coordinates were before.
And even if there is a game with peer-to-peer sword exchange, some adjustment of coordinates would still need to take place in order to account for different sizes of characters and their hand positions.
Do you have another example maybe?
Here's another example for my use-case: a world that is split into several areas: a forest, a desert, mountains, etc. As your characters crosses from the Forest into the Desert, you want to transfer its ownership to the Desert component - but obviously you want this to be transparent to the user.


ESLint 2

Sigh, still a lot of failing tests, I just updated the eslint/espree options to make 50 tests pass but there's still like 20 related to the acorn/espree 3.0 change. Not fun..
Will need to be fixed in the acorn-to-esprima package - although at this point that doesn't make sense at all :sweat_smile:

Thanks for taking care of this.
This snippet is currently still not working with eslint2 and your branch:
const Paragraph = props => {
  const { children, ...rest } = props;

  return (
    <p {...rest}>{children}</p>
  );
};

line 2: children was used before it was defined [no-use-before-define]

Ok since tests pass we can release a beta version for this.


Remove the mesh morphing from the SolidFunctional module

To ensure consistency between physics modules, this PR removes the mesh mutation from the SolidFunctional module. This is OK for output as both ParaView and Sidre provide displace operators.
Resolves https://github.com/LLNL/serac/issues/760

👍

To be clear, now everything is always kept in the reference configuration?

Thanks for looking into this, I was confused by what was going on with all of these mesh transformations in this file, but I wasn't confident enough about what was going on to modify it in the previous solid mechanics update.

👍
To be clear, now everything is always kept in the reference configuration?

Yes - the mesh is now always in the reference configuration. I think in the future it might be nice to register a displacement field with the state manager so a physics developer can grab the deformed configuration if necessary, but we can develop that feature when it is needed.

Codecov Report

Merging #762 (8d62fbf) into develop (1e473ac) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #762      +/-   ##
===========================================
- Coverage    94.51%   94.50%   -0.01%
===========================================
  Files          138      138
  Lines         8550     8519      -31
===========================================
- Hits          8081     8051      -30
+ Misses         469      468       -1

Impacted Files
Coverage Δ

src/serac/physics/solid_functional.hpp
98.37% <100.00%> (-0.21%)
:arrow_down:

src/serac/physics/tests/serac_solid_functional.cpp
99.43% <100.00%> (-0.01%)
:arrow_down:

.../physics/tests/serac_solid_functional_boundary.cpp
100.00% <100.00%> (ø)

...ysics/tests/serac_solid_functional_finite_diff.cpp
100.00% <100.00%> (ø)

...c/physics/tests/serac_thermal_solid_functional.cpp
100.00% <100.00%> (ø)

src/serac/physics/thermal_solid_functional.hpp
56.66% <100.00%> (-1.40%)
:arrow_down:

...merics/functional/tests/functional_comparisons.cpp
97.67% <0.00%> (+0.58%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[MINT-3117] Six pager articles updates

MINT-3117
Description

Created <NeedHelp /> component for both articles and test
Created <ModelSectionCriteriaTable /> component for both articles and test
Created <SimpleList /> component for both articles and test
Updated all 6 Pager content
Updated 2 pager content with shared components
Updated snapshots

How to test this change

Verify matches figma

PR Author Checklist

[ ] I have provided a detailed description of the changes in this PR.
[ ] I have provided clear instructions on how to test the changes in this PR.
[ ] I have updated tests or written new tests as appropriate in this PR.
[ ] Updated the Postman Collection if necessary.

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
When approving a PR, provide a reason why you're approving it

e.g. "Approving because I tested it locally and all functionality works as expected"
e.g. "Approving because the change is simple and matches the Figma design"

Don't be afraid to leave comments or ask questions, especially if you don't understand why something was done! (This is often a great time to suggest code comments or documentation updates)
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.

Few things:

external link icons are now mail envelope icons

The additional resources section still does not match, specially the following:

Model Development Library needs to be changed to Strategy Refresh Model Development Library
2-pager/6-pager repository to be split into 2-pager repository and 6-pager repository
ICIP repository has the wrong URL for its link
PROMs strategy is spelled out to Patient Reported Outcome Measure(PROM) strategy

Lastly, the need help email link does not work
Let me know if you want to sync about this

Ahhh I meant to change it back to Launch Icon.  Thanks for catching this.  I'll have these addressed


Add recent version of TensorFlow

Thank you for this amazing library!
Currently, tf2rl supports only tf_version <= 2.3.0, according to the setup file. Would it be possible to add newer versions of Tensorflow (e.g. 2.4.0)?

Ran bash ./test_run_examples.sh without any errors.

Thanks for creating this issue and checked tf2.4 works!
I just verified it works too. I'll update master branch soon.

Just in case, to reduce workload on your side I've just created a new PR#136.

I merged your PR, and closing this issue. Thanks for your help !


Update aggregation.md

There appears to be a mismatch between the example code and description introduced in c43dc208ac167ea1431baa2d1bf4b3a738fd0404 when the example was fixed the text after the example was not updated.

Thanks for the contribution @strigonLeader !


Incorrect python dependency for pyaml >=23.5.7

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
On conda-forge, this package lists python >=3.6 as a dependency.  However, I installed pyaml 23.5.7 in a python 3.7 environment, and I got an error when trying to import it, as it uses python 3.8 syntax.  Additionally, I see in the project's pyproject.toml file that it has increased its required python version to >=3.8.
Installed packages
N/A

Environment info
N/A

Thanks I'm addressing this in https://github.com/conda-forge/pyaml-feedstock/pull/24


Caption to explain uniform and logarithmic

Is your feature request related to a problem? Please describe.
Technical term which everyone would not know i.e. Logarithmic and Uniform
Describe the solution you'd like
If taking our cursor over it explains what it is or a link to some site which can explain in detail
Describe alternatives you've considered
A link to explain the idea in detail. A small one liner to explain it there itself
Additional context
below image shows the details where we need to see what is this

Would like to work on this. Please assign.

@bootleg-icarus what description will you provide? And where?

@bee-rickey The description would be visible on hover over the switches ( with a delay of say 300ms )
uniform

( if it's off ) Turn on to make Y-axes in all graphs match the same range
(Additional info  - this might make the graphs with comparatively smaller numbers seem flatter )
( while it's on ) Turn off to make Y-axes adjust to their own highest ranges
(Additional info  - this might make all graphs look like they're on a similar trajectory )

logarithmic

( if it's off ) Turn on to make the Y-axes steps jump with powers of 10
( Additional info - if the graph line is going straight up, it still means the numbers are exponentially increasing )
( if it's on ) Turn off to make the Y-axes steps increase linearly
( Additional info - numbers are increasing exponentially if the graph line is curled upwards )

I think the additional info is not necessary but something like that could be viewed when the user clicks a 'see more ' button (or link) on the pop-up.

This sounds good, you can use react-tooltip already present to get this working. A hover over the labels would ideally present this info.


[Feature Request] text2music

Thank you for your great work! I really appreciate your efforts in creating such a wonderful tool for generating music from music. I would like to ask you if you have any plans to fine-tune or train a model checkpoint for text2music. Thank you very much for your time and attention.

They Planned Multi-Lang and then multimodal for images to text as a most likely, text to music seems unlikely

please join discord and let's find someone to work on that :)


Update dependencies + refactor tests

Pull Request Test Coverage Report for Build 334

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 285:
0.0%

Covered Lines:
32

Relevant Lines:
32

💛  - Coveralls


Adapt research-approved praos parameters

@nc6 Do you know what the status on this is?

If people can make actual money on the testnet (due to staking), this must be verified for the testnet release.

@dcoutts do we have sign off on these parameters yet?

@nc6 Can you make a list of parameters that we need? We can then get it verified by @dcoutts and/or the researchers.

@brunjlar: @dcoutts told us you would have some insights on this? Can you update your model with Neil's latest data?
BTW, in case you're interested in the code: https://github.com/input-output-hk/ouroboros-network/blob/master/ouroboros-consensus-shelley/src/Ouroboros/Consensus/Shelley/Protocol.hs.

Final parameters from Lars and Neil:
s = 1s
f = 0.05
valancy = 20
pools 150..250
vast majority of stake in pools
waste: 10, 15%
stability: 3k/f, versus 4k/f

No more changes needed to consensus.


Animation resets if multiple calls are made

Hi,
First of all thanks for this awesome library!
I've been using it and sometimes the graph gets updated twice (usually with the same data), before the animation of the first calls finishes. And this causes the animation to abort and restart for the 2nd call. Which looks odd.
To test this include the following code in the viewDidLoad method of the example project:
  dispatch_after(dispatch_time(DISPATCH_TIME_NOW, Int64(graphView.animationDuration * 0.5 * Double(NSEC_PER_SEC))), dispatch_get_main_queue()){
       self.graphView.setData(self.data, withLabels: self.labels)
 }

I think possible solutions would be:

Always start the 2nd animation at the current point each point is
If the 2nd call has the exact same data as the currently animating call, don't animate it.

Hi!
Thanks for this!
I will have to implement number 1 in the future, but for now I've pushed a new version (1.1.2) which checks if you're setting the same data again and if so, ignores it.
Can you try it out and let me know if it fixes your problem?

@philackm Yeah that fixes it, thanks for the quick reply!

I can back this up, i want to realise a simple WaveForm and it looks uncool, because the whole bar is refreshing again and again. Is there a solution? :/

Can you explain a little bit about what you mean by "a simple waveform"?

Something like this: https://developers.soundcloud.com/assets/posts/waveform_rendering_mask-2d44b706dd9171ec4bdb6720fa2c0fb1.png
But just with Bars going Up. Don't know if i can make it in both directions. Up und Down. Und i have an array which will be reloaded fast. So the animation just startet over and over

I'm still not entirely sure what you want to do.
You should be able to create a bar graph that looks like a waveform, here is an example of a sine wave:

Are you trying to animate the waveform? As in, are you updating the array and then calling setData in very quick succession?
If so, that isn't really supported and was not the idea of calling setData, however, you should be able to do it by turning off the startup animation (and by using the latest version in cocoapods, I had to make a change to make it work). Again, this isn't really supported though. (If it is indeed what you're trying to do.)
graphView.shouldAnimateOnStartup = false
I was able to animate a sine wave by turning that setting off and then calling setData very quickly.

Thank you :)

i also want to get a waveform... so you said you called setData very quickly, how can i do that??


container_collector  hangs in node

container_collector  hangs ,No valuable log information was found except alarm mail.

OpenPAI Environment:

OpenPAI version:0.12.0
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Hardware (e.g. core number, memory size, storage size, GPU type etc.):
Others:

Anything else we need to know:

Logs in the container job-exporter:
WARNING: No swap limit support
WARNING: No swap limit support
WARNING: No swap limit support
WARNING: No swap limit support
2019-05-15 08:03:22,725 - WARNING - container_collector - network.py:162 - unknown format of lsof ['java', '5692', 'root', '537u', 'IPv4', '1874952368', '0t0', 'TCP', '<IP_ADDRESS>:47904-><IP_ADDRESS>:9000', '(ESTABLISHEDlsof:', 'no', 'pwd', 'entry', 'for', 'UID', '1001']
2019-05-15 08:03:43,038 - WARNING - container_collector - network.py:162 - unknown format of lsof ['IPv4', '1875423653', '0t0', 'TCP', '<IP_ADDRESS>:22-><IP_ADDRESS>:53973', '(ESTABLISHED)']
2019-05-15 08:03:43,814 - WARNING - container_collector - network.py:162 - unknown format of lsof ['3u', 'IPv4', '1875442807', '0t0', 'TCP', '<IP_ADDRESS>:22-><IP_ADDRESS>:53976', '(ESTABLISHED)']
2019-05-15 08:03:45,079 - WARNING - container_collector - network.py:162 - unknown format of lsof ['3u', 'IPv4', '1875423653', '0t0', 'TCP', '<IP_ADDRESS>:22-><IP_ADDRESS>:53973', '(ESTABLISHED)']
2019-05-15 08:03:46,389 - WARNING - container_collector - network.py:162 - unknown format of lsof ['3u', 'IPv4', '1875423653', '0t0', 'TCP', '<IP_ADDRESS>:22-><IP_ADDRESS>:53973', '(ESTABLISHED)']
WARNING: No swap limit support
2019-05-15 08:04:04,005 - WARNING - container_collector - network.py:162 - unknown format of lsof ['3u', 'IPv4', '1875442807', '0t0', 'TCP', '<IP_ADDRESS>:22-><IP_ADDRESS>:53976', '(ESTABLISHED)']
2019-05-15 08:04:04,873 - WARNING - container_collector - network.py:162 - unknown format of lsof ['12u', 'IPv4', '1880139804', '0t0', 'TCP', '<IP_ADDRESS>:35232-><IP_ADDRESS>:5010', '(ESTABLISHED)']
2019-05-15 08:04:06,210 - WARNING - container_collector - network.py:162 - unknown format of lsof ['9u', 'IPv4', '1880133129', '0t0', 'TCP', '<IP_ADDRESS>:42186-><IP_ADDRESS>:5010', '(ESTABLISHED)']
2019-05-15 08:04:07,602 - WARNING - container_collector - network.py:162 - unknown format of lsof ['9u', 'IPv4', '1880104773', '0t0', 'TCP', '<IP_ADDRESS>:55796-><IP_ADDRESS>:5010', '(ESTABLISHED)']
WARNING: No swap limit support

@JianfeiHu - are you still seeing the issue today?


Fix broken links in contributor guide.

These modules were recently moved to a new location.

@bors r+

:pushpin: Commit 435994f541cdcf704d35c49169f5dbea43c2e1a2 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 435994f541cdcf704d35c49169f5dbea43c2e1a2 with merge 87871ccfcbc999a829ba119c5b9cba250247afe5...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 87871ccfcbc999a829ba119c5b9cba250247afe5 to master...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 87871ccfcbc999a829ba119c5b9cba250247afe5 to master...

:eyes: Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request.


Create fritzbox-blacklist-495-hostname

Include:
Alexa Metrics
Amazon Ads
Amazon/Alexa Tracking
Apple Metrics
being sold out
Disqus Ads
Facebook Ads
Facebook Tracking, WhatsApp Telemetry
Fastclick
Google AdMob Ads
Google AdWords, AdSense and Analytics
Google DoubleClick Ads
Hosted by Amazon CloudFront
LinkedIn Ads
List of more ads
Malware domains related to browser extensions
Microsoft Ads
Microsoft Tracking
Other ads/tracking/malware
Spotify Ads, Tracking and Analytics
such as NanoDefender and The Great Suspender
Twitter
Yahoo Ads
Yahoo Tracking
Yahoo's AdFlurry
Yahoo's Mydas
YouTube Ads
From:
https://raw.githubusercontent.com/DRSDavidSoft/additional-hosts/master/domains/blacklist/adservers-and-trackers.txt

Nice list, thank you!


: A question and require your guidance on the Course : Advanced Data Science with IBM Specialization on Coursera

I have enrolled for the course Advanced Data Science with IBM specialization and completed Week 1 through 4 partially. https://www.coursera.org/specializations/advanced-data-science-ibm
Please note, I am a new to coding person with only minimal knowledge on Phyton - based on my course on www.cognitiveclass.ai.
In this, on week 2, 3, when try to complete the programming assignments on Spark and tried to set up and learn, I am stuck. Unfortunately, the guidelines video hosted here :  is also not available / working.
I have also posted this query on the discussions forum there in Coursera last week and awaiting advise.
Ref : https://www.coursera.org/learn/ds/discussions/weeks/2/threads/Y40fZ4hDEemB1RILjckZRA
However I am worried that I might loose time in this process.
In fact, I require advise on how and where to start the environment and continue.
Could you please guide and advise me for successful learning of this course.

Answered in the Coursera thread


Initial time

Hello
Thanks for making such a great component.
how can i load Initial just time when I use TimePicker plugin?
I have just string time like (10:22) no date

I had already prepared this example.
It may be useful for you too:
https://codesandbox.io/s/crazy-rubin-2vdpbr?file=/src/App.js

Thank you very much


Id3

confirmed:ID3
DocumentType:verified
Nationality:verified
DeviceDetails:verified
Experience:verified
Screenshot:verified

confirmed:ID3
DocumentType:verified
Nationality:verified
DeviceDetails:verified
Experience:verified
Screenshot:verified


Commvault Cloud for Splunk SOAR

Name of the app
commvaultcloud
Integration
This app integrates with Commvault Platform Security APIs to retrieve information about threats.
About
Introducing the Commvault Cloud Splunk SOAR app. It enables security analysts to swiftly respond to threats using pre-built integrations, and playbooks for securing and auditing the Commvault Cloud platform.
With today's evolving threat landscape, data is under constant risk of data destruction and exfiltration. Organizations are challenged with responding to security events as quickly as they can to limit the impact of cyber threats on their production as well as backup data. This SOAR app allows organizations to monitor anomaly alerts from the Commvault Cloud platform, so they can respond with orchestrated actions to help fortify the data protection platform so they can recover in the event of a cyber incident.
Key features :

Support for Commvault Cloud (software and SaaS)
Suspicious file anomaly monitoring to indicate file encryption.
Fetch Commvault Cloud file anomaly alerts over API
Ability to export and view list of infected files for investigation

Hello,
The repo https://github.com/splunk-soar-connectors/commvaultcloud has been created.


Proto2 Support

Is it expected that the vtproto files aren't currently being generated for proto2 files or do I have something misconfigured?

Nope, this is expected (and I unfortunately forgot to document it :sweat:). I intentionally skipped support for proto2 because Vitess doesn't use it and it makes memory pooling trickier. I take it your project is stuck in proto2 and not planning to upgrade to proto3? I'm still unsure of whether the additional complexity of supporting legacy proto2 is worth it, given the minimal use case.

The specific case is some legacy proto2 messages getting referenced in proto3 messages which causes compile errors as the vtproto helpers are not generated but are expected to exist in the proto3 helpers.

As an additional entry point - our codebase does still have hundreds of proto2 files and the migration would be tricky and often risky.
This makes me interested if you're willing to accept patches to support generation (un)marshal/size methods for proto2?
I understand that pooling logic is tricky and we could skip it & ask people to migrate to proto3 for this additional feature.

@euroelessar: sure thing. It actually shouldn't be a massive change, some of the generators already have a proto3 param that toggles between proto2 and proto3 generated code. It's just a matter of finishing up all the corner cases. If you send a PR I'll review it.


Add option to make it impossible to overwrite mapped exception classes

It's currently possible to override any exception handler. This makes it hard to implement redirects using exceptions, and also potentially hurts plugin authors who want to guarantee that their plugin works as expected. We could add something like
app.exception(class, canBeOverriddenBoolean, handler)

After adding a special rule to the except mapper, this is no longer relevant for redirects.

I'll close this for now, can always be added later.


Invariants held by diasllowing enum discriminants

This is more of a question, which I was wondering about,
If one tries to set a discriminant on a Token e.g.
enum Token { #[end] End = 0 }

Logos will produce:

error: Token::End has a discriminant value set. This is not allowed for Tokens.

When converting token to a numerical type with as, the tokens appear to sequence 0, ...
But i haven't tracked down anything which ensure this yet.
Is logos by disallowing ensuring an invariant that the derived enum is 0 based and sequential?
If so, is this something that could be relied upon as part of its public API?

An enum with only unit variants, will have discriminants set by default, counting from 0 in order they are defined. You can see this is you do the following:
enum Foo {
    Bar,
    Baz = 0,
}

Produces error:
error[E0081]: discriminant value `0` already exists
 --> src/lib.rs:3:11
  |
2 |     Bar,
  |     --- first use of `0`
3 |     Baz = 0,
  |           ^ enum already has `0`

The rule to disallow discriminants is predetermined on the existence of the lookup! macro, it uses the SIZE associated const of the Logos trait to figure out a size of a lookup table that needs to be constructed, and allows you to index into it by casting the token enum as usize.
In principle we could relax this constraint, all that needs to be done to make sure that SIZE equals largest discriminant set (either explicit or implicit) + 1.

I only really wanted to ensure the first was zero,
enum Foo {
    Bar = 0,
    Baz,
}

perhaps an error check like the following?

        if Some(0) != variant.discriminant {
            panic!(
                "`{}::{}` has a non-zero discriminant value set. This is not allowed for Tokens.",
                name, variant.ident
            );
        }

Gonna constraint it to the number of variants, that's more relaxed than straight out disallowing setting the values while not having any unforeseen consequences. Will push to rewrite in a minute:
error: `Token::Error` discriminant value is too large. Value should be smaller than number of variants (20).
 --> tests/tests/advanced.rs:7:13
  |
7 |     Error = 1000,
  |             ^^^^

Pushed.

@ratmice I've been refactoring some code, found this check, and recalled talking about relaxing this further. If there isn't an open issue for this and you still need this, open one and I'll flag it for 0.11.1.

@maciejhirsz I still need to do some refactoring before I can fully make use of the existing relaxation above in my program (for reasons not relevant to logos).  So at this point I'll leave that decision up to you?

I forgot to mention in in release notes, but I already did relax it just now in 0.11.1. If you have 3 variants, and you set discriminant = 9 to one of them, T::SIZE will automatically bump to 10 without any errors.
So, you should be good to go :)


Issue with accessing LibreBoard on mobile web browser

When accessing LibreBoard on mobile web browser, the UI gets all mixed up

Old UI works decently on mobile for me, but newui is unusable


[eslint-plugin-lit-a11y]: refactor to compose rules, test settings

Composing Rules
Currently, we copy logic into each rule to verify that the file imports lit-html's html, This is great for users but hard to maintain, as we have to remember to copy in the logic for each rule.
We can compose rules with https://github.com/not-an-aardvark/eslint-rule-composer
so, instead of adding a predicate in each rule's AST visitor, we can just compose them together:
const { filterReports } = require('eslint-rule-composer');
const eslint = require('eslint');
const hasLitHtmlImport = require('../utils/hasLitHtmlImport');

const TheRule = { /*...*/ }

module.exports = filterReports(TheRule, hasLitHtmlImport);

Much DRYer.

Applying settings in tests
In tests, we map over each test case to add an import statement for lit-html's html. We can apply settings for a test run with:
const ruleTester = new RuleTester({ settings: { myPluginSpecificSetting: ['lol'] } });

we could make litHtmlSources opt out by accepting false as an option, then disable checking for tests with
const ruleTester = new RuleTester({ settings: { litHtmlSources: false } });

https://www.npmjs.com/package/eslint-rule-extender is a maintained alternative to eslint-rule-composer

from @kaicataldo, maintainer on eslint-rule-extender, when asked if we can stop a rule from reporting errors in one visitor from another (shared) visitor:

yes - in the case of eslint-rule-extender, you’d do this by checking the reports and filtering them out. Have you looked at https://eslint.org/docs/rules/no-restricted-syntax or https://eslint.org/docs/rules/no-restricted-imports? Obviously don’t have much context here, but that might get you what you need

ostensibly we'd use reportOverrides for that

If you're creating an additional ImportDeclaration visitor, I believe you should be able to either:

Check the error message or analyze the node that's in the report metadata passed to reportOverrides
Add some data in the metadata (i.e. { isLitHtml: true }) that is checked in reportOverrides. This data is generally intended for interpolation into the error message itself, but I don't think (not 100% sure without checking the code) that we enforce that all data is used in the error message. This could help avoid extra calculation overhead.

I'm going to take a crack at this today

this seems to work, without requiring any changes to rule code except for applying ruleExtender.
preparing a PR
const HasLitHtmlImportRuleExtension = {
  /**
   * @param {import('eslint').Rule.RuleContext} context
   * @return {import('eslint').Rule.RuleListener}
   */
  createAdditionalVisitors(context) {

    /** if user set `litHtmlSources` to literal `false`, analyse all html-tagged literals */
    const userExplicitlyDisabledImportValidation =
      context.settings.litHtmlSources === false;

    /* eslint-disable no-param-reassign */

    context.parserServices.shouldAnalyseHtmlTaggedLiterals =
      userExplicitlyDisabledImportValidation

    return {
      ImportDeclaration(node) {
        const shouldAnalyseNode =
          userExplicitlyDisabledImportValidation ||
          hasLitHtmlImport(node, context)

        if (shouldAnalyseNode)
          context.parserServices.shouldAnalyseHtmlTaggedLiterals = true
      },
    };

    /* eslint-enable no-param-reassign */
  },

  /**
   * @param {import('eslint').Rule.RuleMetaData} meta
   * @param {import('eslint').Rule.RuleContext} context
   */
  reportOverrides(meta, context) {
    const { shouldAnalyseHtmlTaggedLiterals } = context.parserServices || {};
    return !!(shouldAnalyseHtmlTaggedLiterals);
  },
}

ok my solution relies on the ImportDeclaration visitor in createAdditionalVisitors running before the subclass rule, I hope that's a good assumption

@bennypowers I believe that the subclass rule's visitors will always run first (unless there's a bug in the code). ESLint's :enter:/:exit visitors are intended to be used if you have state you need to manage.
I hadn't considered what would happen with multiple :enter:/:exit visitors, but I think I'll punt on that for now since it seems unlikely!


Edit state context issue.

I am not sure why but recently there has been context issues with the edit state provider. I am assuming because the edit state being "built" and the one being used internationally are separate. (This might be because we switched to ES build @jeffsee55 ?)
This means that when the user uses the edit state in their app (for admin page, an edit button etc) it works as expected but when used internally we get the wrong context.
For example when doing this the tinacms package. We get the wrong context.
import { useEditState } from '../edit-state'
//...

const { setEdit } = useEditState()

I have been trying to fix this for a while now and I am sorta blocked.
What I have tried so far to fix it
1. Different imports
If the problem is the that two different builds are happening then importing from the correct file should fix it.
I tried
import { useEditState } 'tinacms/dist/edit-state'

but got this error
seEditState' is not exported by dist/edit-state.js, imported by src/auth/TinaCloudProvider.tsx
file: /Users/logananderson-forestry-mac/dev/tinacms/tinacms/packages/tinacms/src/auth/TinaCloudProvider.tsx:6:9
4: import { useTinaAuthRedirect } from "./useTinaAuthRedirect";
5: import { createClient } from "../utils";
6: import { useEditState } from "tinacms/dist/edit-state";
            ^
7: function sleep(ms) {
8:   return new Promise((resolve) => setTimeout(resolve, ms));
Error building tinacms
(node:49488) UnhandledPromiseRejectionWarning: Error: Error: 'useEditState' is not exported by dist/edit-state.js, imported by src/auth/TinaCloudProvider.tsx

I also tried
import { useEditState } from 'tinacms/dist/edit-state.es'

Which does not produce a build error but the context run time error still arises.
Error: No `TinaEditProvider` found

I even tried changing the import in tina-cloud-stater app to
import { TinaEditProvider } from "tinacms/dist/edit-state.es";

with no luck.
1. Different build command
I also tried a few different build commands in combination with the different imports.
different order
  "build": "yarn tsup src/edit-state.tsx  src/index.ts --format cjs --dts",
  "watch": "yarn tsup src/edit-state.tsx src/index.ts --watch --format cjs --dts"

Got the same context error
Exclude tinacms
"build": "yarn tsup src/edit-state.tsx  src/index.ts --external tinacms --format cjs --dts",
"watch": "yarn tsup src/edit-state.tsx src/index.ts --external tinacms --watch --format cjs --dts",

Got the same context error.
Next steps
Not sure what else to try. I could also try putting the edit-state in a separate package? Not sure if this is the end result we want though.

I tried moving the edit-state into its own package and it did resolve the context issue.
You can see a draft PR here: https://github.com/tinacms/tinacms/pull/2104

@jeffsee55 it strangely was working before.
We need to have the context be available to us as well as the end user because we need to do things like

Exit edit state when they close the login model
We might want to provide the admin page for the user at some point
then there admin.tsx would look like

import { AdminPage } from "tinacms"

export default AdminPage

Even when in the tinacms package I tried importing from the dist folder and that also did not work.

There are definitely some problems with this, but it's working for now - though the details aren't the best. As we're looking into studio this will likely be part of a UI that we own.

Discussing out of band and we may introduce a "context" package that's used as a direct dependency by tinacms. Some discovery needs to be done on this

Should be fixed by https://github.com/tinacms/tinacms/pull/2281


WOFF2 font please

Smaller, & soon to be compatible with 1/2 of the (global) in-use browsers.
Let me know if you want me to make a quicky conversion.

Pull requests are greatly welcomed. Our current WOFF is 1.3KB (non-gzipped).
Thanks. Tom!

This issue spurred the base64 embedding of the WOFF into the actual CSS which should be more performance friendly (at least untill HTTP/2 comes out!). See #1291.


Pandora's Box

The game works, but there are two puzzles issues that can cause the game to crash. The game has the official patch which is mandatory for Windows 10/11.
The first one is that if you put the last piece at 90% the game crashes but if you save, exit the game and resume the game and put the last piece the game doesn't crash

the other puzzle, if you rotate it in rapid succession the game crashes

The game does not crash for me, but I also doubt that this has anything to do with DDrawCompat.
Enable LogLevel=debug and CrashDump=full, and upload both the log file and the crash dump.

The game crashes but it doesn't create any dump file but only a log file and it weighs 991 mbs! I uploaded the log to mega https://mega.nz/folder/PEkWEBbK#_BVMadddtBn3VVL88ocooA

Well, it doesn't crash. After the last WM_LBUTTONUP handler executes, the game seems to exit gracefully, destroying all windows, restoring the display mode, etc. But I don't see why, there is nothing suspicious in the logs. Must be something weird going on in the game logic.

Well, it doesn't crash. After the last WM_LBUTTONUP handler executes, the game seems to exit gracefully, destroying all windows, restoring the display mode, etc. But I don't see why, there is nothing suspicious in the logs. Must be something weird going on in the game logic.

So I made a video, apparently it tends to crash (instantly) when I save after starting the puzzle
https://github.com/user-attachments/assets/1aa24b4b-5855-4331-9a0f-0ff5df389eb3
Log
https://mega.nz/folder/PEkWEBbK#_BVMadddtBn3VVL88ocooA

Ok, I can reproduce this one. It's a read access violation in pandora.exe. Happens without DDrawCompat too, and has no relation with any DirectDraw operations either, so it's out of scope for this project.

Ok, I can reproduce this one. It's a read access violation in pandora.exe. Happens without DDrawCompat too, and has no relation with any DirectDraw operations either, so it's out of scope for this project.

So this is a bug, no need to save during that puzzle, got it


Escape key should only close the treecrumb (and other elements) UPDATE BOOTSTRAP

For the time being, the escape key closes the elements too.

reviewed


Error loading walkthrough content on vscode.dev

Error loading webview: Error: Could not register service workers: NotSupportedError: Failed to register a ServiceWorker for scope ('https://05g7dv7ef9i6vq72pvc86mfaee1k271hhhm8tnga2o1mm63c41ql.vscode-cdn.net/stable/6261075646f055b99068d3688932416f2346dd3b/out/vs/workbench/contrib/webview/browser/pre/') with script ('https://05g7dv7ef9i6vq72pvc86mfaee1k271hhhm8tnga2o1mm63c41ql.vscode-cdn.net/stable/6261075646f055b99068d3688932416f2346dd3b/out/vs/workbench/contrib/webview/browser/pre/service-worker.js?v=4&vscode-resource-base-authority=vscode-resource.vscode-cdn.net&remoteAuthority='): The user denied permission to use Service Worker..

https://user-images.githubusercontent.com/25163139/201399159-4c68035e-4043-4ac5-ac8b-8d2c18c94b13.mp4
cc @digitarald @bamurtaugh perhaps?

@daviddossett which browser is this? Maybe duplicate of https://github.com/microsoft/vscode/issues/142813

Interesting - I'm not able to repro on Chrome. Maybe @digitarald has an idea.

We had some iframe/SW related issues in the past that @mattbierner looked into.


[HIVEMALL-42][DOC] Fix the link to license file in README.md.

What changes were proposed in this pull request?
Fix a dead link to the license file.
What type of PR is it?
Documentation
What is the Jira issue?
https://issues.apache.org/jira/browse/HIVEMALL-42
How was this patch tested?
Clicked the link at https://github.com/aajisaka/incubator-hivemall/blob/HIVEMALL-42/README.md

@aajisaka We welcome your first contribution!

Thank you for your quick response.

Coverage increased (+0.3%) to 36.14% when pulling 9e3ec53707f19685954245598cacae6b5690bd13 on aajisaka:HIVEMALL-42 into c4d4511d599dff2ac0795076741738fd86c1bd5c on apache:master.

Coverage increased (+0.3%) to 36.14% when pulling 9e3ec53707f19685954245598cacae6b5690bd13 on aajisaka:HIVEMALL-42 into c4d4511d599dff2ac0795076741738fd86c1bd5c on apache:master.


identified as "win.trojan.pistacchietto"?

Since about a week ago, everything associated with the current version of Platypus is being flagged by my antivirus software as "win.trojan.pistacchietto" and blocked from running. This includes php apps that I build with platypus, as well as new copies of platypus that I download.
Is this an actual infection, or a false positive?
The antivirus software that I'm using is CISCO Amp for Endpoints, it's based on ClamAV(osx.cvd 295)

False positive. All unsigned Platypus-generated apps obviously have the same binary (and hence same fingerprint). In the past, some people have created malicious apps with Platypus, this may cause some simplistic virus software to falsely identify every Platypus binary as malicious.


[AutoPR monitor/resource-manager] Add dynamic metric alert api

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5677

(message created by the CI based on PR content)

Codecov Report

Merging #4896 into restapi_auto_monitor/resource-manager will increase coverage by 0.66%.
The diff coverage is 47.22%.

@@                            Coverage Diff                            @@
##           restapi_auto_monitor/resource-manager    #4896      +/-   ##
=========================================================================
+ Coverage                                  48.84%    49.5%   +0.66%
=========================================================================
  Files                                       6865     6341     -524
  Lines                                     195670   177310   -18360
=========================================================================
- Hits                                       95566    87777    -7789
+ Misses                                    100104    89533   -10571

Impacted Files
Coverage Δ

...rt_single_resource_multiple_metric_criteria_py3.py
62.5% <ø> (ø)
:arrow_up:

...e/mgmt/monitor/models/multi_metric_criteria_py3.py
42.85% <16.66%> (-23.81%)
:arrow_down:

...mgmt/monitor/models/dynamic_metric_criteria_py3.py
45.45% <45.45%> (ø)

...mgmt-monitor/azure/mgmt/monitor/models/__init__.py
52.98% <50%> (-0.05%)
:arrow_down:

...or/models/dynamic_threshold_failing_periods_py3.py
62.5% <62.5%> (ø)

.../mgmt/monitor/models/static_metric_criteria_py3.py
55.55% <66.66%> (ø)

...e-mgmt-netapp/azure/mgmt/netapp/models/__init__.py
60% <0%> (-12.92%)
:arrow_down:

...azure/mgmt/eventhub/v2017_04_01/models/__init__.py
58.82% <0%> (-12.85%)
:arrow_down:

...edia/azure/mgmt/media/models/job_input_clip_py3.py
60% <0%> (-11.43%)
:arrow_down:

...ervicebus/azure/mgmt/servicebus/models/__init__.py
58.44% <0%> (-10.88%)
:arrow_down:

... and 614 more

Continue to review full report at Codecov.

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

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/4563