feat: fixed incorrect issuer for certificates & added x509 prints

This pr implments #351
See also:
https://github.com/iqlusioninc/yubikey.rs/pull/352
(I have just re-opened it, cause it was easier then to re-base everything)

@Scarjit needs rustfmt and looks like a couple clippy nits


Update building-the-node-using-nix.md

The install script needs to be executable.

bors r+

Thanks, subsumed by: https://github.com/input-output-hk/cardano-node/pull/2442


systrace not working

C:\Users\user\Downloads>python %ANDROID_HOME%/platform-tools/systrace/systrace.p
y --time=10 -o %userprofile%/trace.html gfx view res
Starting tracing (10 seconds)
Tracing completed. Collecting output...
Traceback (most recent call last):
File "C:\Users\user\AppData\Local\Android\sdk/platform-tools/systrace/systrace
.py", line 49, in
sys.exit(run_systrace.main())
File "C:\Users\user\AppData\Local\Android\sdk\platform-tools\systrace\catapult
\systrace\systrace\run_systrace.py", line 196, in main
main_impl(sys.argv)
File "C:\Users\user\AppData\Local\Android\sdk\platform-tools\systrace\catapult
\systrace\systrace\run_systrace.py", line 191, in main_impl
controller.StopTracing()
File "C:\Users\user\AppData\Local\Android\sdk\platform-tools\systrace\catapult
\systrace\systrace\systrace_runner.py", line 48, in StopTracing
self._tracing_controller.StopTracing()
File "C:\Users\user\AppData\Local\Android\sdk\platform-tools\systrace\catapult
\systrace\systrace\tracing_controller.py", line 183, in StopTracing
self._IssueClockSyncMarker()
File "C:\Users\user\AppData\Local\Android\sdk\platform-tools\systrace\catapult
\systrace\systrace\tracing_controller.py", line 239, in _IssueClockSyncMarker
agent.RecordClockSyncMarker(sync_id, ControllerAgentClockSync)
File "C:\Users\user\AppData\Local\Android\sdk\platform-tools\systrace\catapult
\systrace\systrace\tracing_agents\atrace_agent.py", line 229, in RecordClockSync
Marker
shell.RunCommand(cmd, close=True)
File "C:\Users\user\AppData\Local\Android\sdk\platform-tools\systrace\catapult
\systrace\systrace....\devil\devil\android\sdk\adb_wrapper.py", line 213, in R
unCommand
return (result[:-1], int(result[-1]))
ValueError: invalid literal for int() with base 10: ''
Windows 8
Also there was also an import error on pywin. I resolved it with
pip install pywin32
pip was a problem too. It was not working,
So I solved that problem by setting path C:\Python27\Scripts into Environment Variables

Same issue0 ValueError: invalid literal for int() with base 10: '' any news on this?


Experimental support for argocd action to deploy the argo app

This is a WIP to make the basic template use the argocd create-resources action so
if RHDH have the argocd scaffolder plugin enabled and configured we can immediately deploy
that argocd app.
The result should be a workflow  which is immediatly built and deployed and after deployment available in the orchestrator overview page.
Signed-off-by: Roy Golan<EMAIL_ADDRESS>

@rgolangh when this one verified, it closes https://issues.redhat.com/browse/FLPATH-1322
Pls don't hesitate to assign it to you :-)

@rgolangh please take into account also the need to access private repos, for which I initially raised
https://github.com/RoadieHQ/roadie-backstage-plugins/issues/1298

@masayag @ElaiShalevRH @gabriel-farache do we want to take over this PR?

@masayag @ElaiShalevRH @gabriel-farache do we want to take over this PR?

I don't think we have the capacity to handle this issue, also it requires https://github.com/RoadieHQ/roadie-backstage-plugins/issues/1298 to be solved.


Conversion to float in freesurfer.parcels_to_vertices

Here's a very minor issue that annoyed me this morning :
In freesurfer.parcels_to_vertices, we might want to handle cases where the parcellated data to be projected on the vertices are int values since np.nan cannot be inserted into numpy arrays of int. We could for example convert ints to floats. Without proper handling of this case, the user might be confused as to what goes wrong when the "cannot convert float NaN to integer" ValueError is raised.

Seems reasonable! Should be as straightforward as modifying:
https://github.com/netneurolab/netneurotools/blob/4ff61eb5452f90a5366a289c91c9ad2ed0fd2940/netneurotools/freesurfer.py#L279
to:
data = np.vstack(data).astype('float')

Feel free to open a PR with the changes 🙌
(Alternatively, instead of casting ourselves we could just raise a more helpful error, i.e., if an int array is detected let the user know that they need to cast to float to use the function. Your choice!)


LLD - bug - fixes loading indicator on swap page

📝 Description
In LLD, the loading indicator is currently misaligned in the From form on the Swap screen.
❓ Context

Impacted projects:  LLD
Linked resource(s):  https://ledgerhq.atlassian.net/browse/LIVE-4747

✅ Checklist

[ ] Test coverage
[ ] Atomic delivery
[ ] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

LGTM


CLI hangs when using Node 18

When using Node 18, the CLI hangs when running basic commands that reach out to the server. Switching to Node 16 or 14 resolves the error.
Steps to reproduce:

Install and configure Akamai CLI and install property-manager package
run the following (assuming nvm is used to manage node versions)

nvm use 16
akamai property-manager list-groups
nvm use 18
akamai property-manager list-groups

Expected results: CLI works on both node versions.
Actual results: CLI only works on pre-18 versions.
Notably, if running with the -v flag, it will show the underlying error - a 403 (detail: "Client not authorized"), however on Node 16 it works correctly.
Similar issue in edgeworkers package: https://github.com/akamai/cli-edgeworkers/issues/110

Similar as noted on the linked edgeworkers package, things seem to be working now (no changes to node/akamai/property-manger versions). Perhaps a server-side change went out? Either way, closing this


⚠ Secret Website 3 has degraded performance

In a8e0458, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 1954 ms

Resolved: Secret Website 3 performance has improved in 2e1a7bb.


[SPARK-21447][WEB UI] Spark history server fails to render compressed

inprogress history file in some cases.
Add failure handling for EOFException that can be thrown during
decompression of an inprogress spark history file, treat same as case
where can't parse the last line.
What changes were proposed in this pull request?
Failure handling for case of EOFException thrown within the ReplayListenerBus.replay method to handle the case analogous to json parse fail case.  This path can arise in compressed inprogress history files since an incomplete compression block could be read (not flushed by writer on a block boundary).  See the stack trace of this occurrence in the jira ticket (https://issues.apache.org/jira/browse/SPARK-21447)
How was this patch tested?
Added a unit test that specifically targets validating the failure handling path appropriately when maybeTruncated is true and false.

Can one of the admins verify this patch?

ok to test

Test build #79714 has started for PR 18673 at commit caeac28.

Test build #79714 has finished for PR 18673 at commit caeac28.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79714/
Test FAILed.

Test build #79720 has started for PR 18673 at commit 484141a.

Test build #79720 has finished for PR 18673 at commit 484141a.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79720/
Test FAILed.

Merged build finished. Test FAILed.

Test build #79731 has started for PR 18673 at commit b82847e.

Test build #79731 has finished for PR 18673 at commit b82847e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79731/
Test PASSed.

Merging to master / 2.2.


Computing covariance matrix

Hi again
I am experimenting issues with covariance matrix computation. In some tests, I have used tf$Session()$run(kernel$K(X)) which works just fine, but there are "helpers" that are more user oriented that do not work for now:
library(gpflowr)

# just fine
X <- matrix(runif(10),10,1)
X2 <- matrix(runif(12),12,1)
k <- kernels$RBF(1)

#errors
K <- k$compute_K(X,X2)
K <- k$compute_K_symm(X)
Kdiag <- k$compute_Kdiag(X)

the output for the first two errors is
Error in eval(substitute(expr), envir, enclos) :
  ValueError: as_list() is not defined on an unknown TensorShape.

and for the last one, error says that k$compute_Kdiag is not a function.
Any thoughts?

Hi N,
Yup, I didn't manage to get autoflow (the clever bit that overwrites functions to run via tensorflow, caching data) working yet. That's my focus for this morning, will let you know how I get on!

This took a bit longer than I'd hoped, but autoflow is now working on master, and I registered the autoflowed Kdiag method. I get the following on a modified version of the MWE:
library(gpflowr)
X <- matrix(runif(3), ncol = 1)
X2 <- matrix(runif(4), ncol = 1)
k <- kernels$RBF(1)

k$compute_K_symm(X)

          [,1]      [,2]      [,3]
[1,] 1.0000000 0.9914959 0.9611905
[2,] 0.9914959 1.0000000 0.9186081
[3,] 0.9611905 0.9186081 1.0000000

k$compute_K(X, X2)

          [,1]      [,2]      [,3]      [,4]
[1,] 0.9046513 0.9214427 0.9132653 0.8589681
[2,] 0.8459843 0.9632060 0.8564646 0.9153044
[3,] 0.9862657 0.7904032 0.9895977 0.7069812

k$compute_Kdiag(X)

[1] 1 1 1

nullified by #27


Point docs to docs.rs

The docs under http://docs.piston.rs/ are out of date and people are not using them because of docs.rs is more convenient. I think we should point the links over to docs.rs.
Is there a way to point to the latest crate version on docs.rs?

I figured it out: You can link to e.g. "docs.rs/piston" and it will redirect to the latest version, e.g. "https://docs.rs/piston/0.32.0/piston/".
We could generate a TODO-list from Eco and publish minor updates.

This is now done for the Piston core.


Custom theme breaks colours and interactivity

I created a custom theme using the theme builder:
  daisyui: {
    themes: [
      {
        whythawk: {
          primary: "#3A5C84",
          "primary-focus": "#6D8FB7",
          "primary-content": "#ffffff",
          secondary: "#5C843A",
          "secondary-focus": "#8FB76D",
          "secondary-content": "#ffffff",
          accent: "#C43900",
          "accent-focus": "#E88C66",
          "accent-content": "#ffffff",
          neutral: "#2a2e37",
          "neutral-focus": "#8d929a",
          "neutral-content": "#ffffff",
          "base-100": "#ffffff",
          "base-200": "#f9fafb",
          "base-300": "#d1d5db",
          "base-content": "#1f2937",
          info: "#4974a5",
          success: "#689441",
          warning: "#ff99900",
          error: "#da4000",
        },
      },
    ],
  },

I made no other changes and followed the installation guide exactly. My stack is:

Nuxt 2.15.8
DaisyUI 1.14.0
Tailwind v2.2.9

And nothing works as expected. Here is what I see:
Primary button:

Primary button hover:

Code for that button:
<button
  type="submit"
  class="btn btn-primary"
>
Create
</button>

This has neither of the effects of the standard or outline buttons:

Toggle true state:

Toggle false state:

Code for the toggle:
<input
  type="checkbox"
  class="toggle toggle-sm"
/>

And none of the transitions or hover effects work. Something is blocking or interfering, but I've no idea what to look for. There are no error messages or anything to hint at what may be going wrong.
What does happen - sometimes as the page is loading - is that I do see the elements in the theme colours I assigned, but once the page loads fully all the custom themes and default behaviours are lost.
I'd appreciate any suggestions.

@saadeghi I've invited you to the repo. Sorry, it's still a major work-in-progress and I haven't committed the changes made at your suggestion. I deleted forms, as well as a few legacy bits of css that were hanging around. I'm not sure what else to delete.


ERROR: Directive 'jinja' cannot be mocked: MockingError: MockStateMachine has not yet implemented attribute 'insert_input'

Hi, thanks for the nice work! I found sphinx-jinja2 very useful when I needed to customise the generated documentation.
Since the generated documentation contains markdown syntax, I try to use myst-parser with sphinx-jinja2, but encountered the following error:

I’d be really grateful if you had any suggestions that could help me with this issue!

The template output is fed back into the RST parser. When the source uses MyST parser (please confirm), this error will be triggered:
   Directive 'jinja' cannot be mocked: MockingError: MockStateMachine has not yet implemented 'insert_input'.

The workaround is to wrap the jinja directive inside rst-eval
```{rst-eval}
   .. jinja::
      :ctx: {"foo":"bar"}

      Baz.
```

rather than using it directly:
```{jinja}
   :ctx: {"foo":"bar"}

   Baz.
```


fix(date-field): corrects date display on IE11

This explicitly set width to 100% which fix this issue
Description
The date fields don't display properly on IE11. This change corrects the
issue
Fixes
set width to 100% on actual input element. Parent element width doesn't reflect on IE11
Storybook link:
https://deploy-preview-495--legal-and-general-canopy.netlify.app/?path=/story/components-form-date-input--standard
Screenshot:
Before fix

After Fix

Checklist:

[x] The commit messages follow the convention for this project
[ ] I have provided an adequate amount of test coverage
[ ] I have added the functionality to the test app
[ ] I have provided a story in storybook to document the changes
[ ] I have provided documentation in the notes section of the story
[ ] I have added any new public feature modules to public-api.ts

I spoke to the Product Team, got confirmation that we don't need to support IE11. I will go ahead and close PR.


Feature request: Task that preload's scala to target emulator

It would be handy to have a Task included with the android-plugin that could dex and install scala to the emulator.
http://zegoggl.es/2011/07/how-to-preinstall-scala-on-your-android-phone.html

yes that would be cool, great idea.


Implement podman.containers.commit()

Add API support for Commit()
Update tests

Signed-off-by: Jhon Honce<EMAIL_ADDRESS>

Travis mostly looked like flakes, but there was one wierd podman run test failure. Restarted the job to see if it's a fluke.

LGTM

@mheon @umohnani8 @TomSweeneyRedHat @baude PTAL

LGTM
@rh-atomic-bot r+


Clarify reward currency in REWARD page

Following up from here https://github.com/kodadot/nft-gallery/issues/3356
In https://github.com/kodadot/nft-gallery/blob/main/REWARDS.md , it is not clear in the table that the reward amount is in USD (and not KSM since it is your favorite currency)
Describe the solution you would like
I would like to see the currency clearly stated, like in the label
Describe alternatives you have considered
Put the KSM amount ^^
Screeenshots
No response
Are there opened related issues?
No response
Thanks to @joelamouche

@thea-exe


Custom chat GUI support

This pull request is a part of a feature request found here, please read that before accepting this pull request!
This would implement the EnumItems ChatWindow and ChatBar into the chat GUI. These need to be added to the API first which is explained on the feature request.

Pull requests are for fixes to bugs in the core scripts. If you want to suggest new APIs or features then please use the Roblox forums.


TinyMCE with Bootstrap modal

Hi,
I didn't invent this fix by myself, but I report this issue over here, because I think that this could be fixed inside this directive.
Issue is that when you initialize tinymce editor inside Bootstrap modal window and you open tinymce link editor, you cannot add link because you cannot get focus on that link dialog.
Here is discussion of that issue as well coulple of jsfiddle links where the issue is shown and fixed.
http://stackoverflow.com/questions/18111582/tinymce-4-links-plugin-modal-in-not-editable
Here is the fix from stackoverflow
$(document).on('focusin', function(e) {
    if ($(e.target).closest(".mce-window").length || $(e.target).closest(".moxman-window").length) {
        e.stopImmediatePropagation();
    }
});

I know that this is a framework specific issue, but I think that if there would be generic solution which doesn't break anything else, that would be best. But there could also be some property like 'data-is-bootstrap' added to directive which could be used to add that fix....

This has a problem - if a user has multiple tinymce windows, then it may not target the desired window.
I also recommend investigating using a library like UI Bootstrap or Angular Strap over vanilla Bootstrap JS.

Closing this issue due to age. If you are still having trouble please open a new issue with a fiddle attached.


[Doc] Improve S3File Source & S3File Sink document

sub task of https://github.com/apache/seatunnel/issues/4572

LGTM It would be better to fix these typos.

Thanks, I finished it.

@EricJoy2048 Conflicts need to be resolved.

@EricJoy2048 Conflicts need to be resolved.

done


Make the dashboard service wait until the db is available.

Description of changes:
This commit introduces a script (wait-for) in the Dashboard Dockerfile to delay the dashboard startup until the Postgres db is available. It is already added to the docker-compose.yml.
Additionally, I reordered some Dockerfile statements to leverage the implicit Docker layer cache. This allows a better roundtrip when developing locally.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I also introduced more symbolic configuration of networks and volumes.

The docker-compose.yml now allows to pass GH_ACCESS_TOKEN and OCTOKIT_API_ENDPOINT from the user's environment instead of setting it to an empty value.

Thank you Tobias! :)
It looks good. I want to do a manual test before merging, and I also need to look at gesellix/wait-for to see what that is licensed under.

Noting that wait-for is MIT licensed.
My Docker inexperience is a bit confused though, what happens when the current FROM ruby and the FROM geelix are providing different linuxes? Alpine vs Debian.

@hyandell The multiple FROM statements are a feature called multi-stage builds, where you more or less can describe independent builds inside your Dockerfile. The only "shared" resource is the filesystem, which enables you to copy files from one image's filesystem to another image. The different platforms are only relevant when copying binaries compiled for different platforms (or against different dynamic libs). In this special case the wait-for lib is only a shell script, so the platform isn't relevant.
While writing this I become aware that we could simply replace the multi-stage build with a RUN statement in which you only download the wait-for script. If you prefer such a solution, just leave a note here and I'll change it.

Please also note that I'm not the actual author of the wait-for script, but only forked from https://github.com/Eficode/wait-for, to add an enhanced Dockerfile.

Now regarding the license: would wait-for with MIT be an issue?

MIT is good. Doing the simpler RUN statement sounds better to me.

@hyandell alright, done :)
If you prefer, I can also squash the commits.

Thanks Tobias.
I did a bit of chatting internally; the one thing that stands out is the curl -o for the wait-for code. This gives your repo a very direct injection point to any of these docker instances being setup, whereas the rest of it is based on trust in Debian's release processes.
What do you think to copying the shell script in as part of the patch? Also, you should probably have an MIT license on the top of the file.

@hyandell You're right, using curl breaks the current barrier of trust. I fully agree that including the script in this repo makes sense. As I mentioned, I'm not the actual author of that script, so do we need to ask the people at https://github.com/Eficode/wait-for?
I'm not very experienced in such legal aspects, but technically I can certainly add the wait-for script in the patch and also include its license at its top.

Thanks @gesellix - putting the license at the top like that is perfect. The original is MIT, so that fits.

PR merged :)

thanks :)


Bug(???) in web and matlab version

CSVtable.zip
Hello, could you possibly tell me what's wrong with these files  ?
CSV generated from table by writetable(Csvtable,CSVfilename,'Delimiter',',')
Error Message in matlab :
Error using bootci>bootbca (line 265)
With 'type' equal to 'bca' (the default), BOOTCI uses JACKKNIFE internally. The call to JACKKNIFE failed with
function '@(x)nanmean(x)'. Try using a confidence interval type other than 'bca'.
Error in bootci (line 171)
[ci,bootstat] = bootbca(obsstat,nboot,fun,alpha,weights,bootstrpOptions,data{:});
Error in bootmoes (line 15)
bci=bootci(10000, hm, vec)';
Error in FscatJit2 (line 170)
[av(idx), moes, bci] = bootmoes(curDat);
Error in dabest (line 19)
[ss] = FscatJit2(identifiers, data);
Caused by:
Error using jackknife
Too many input arguments.

I'm having the same issue, I'll open a new one.


Serve stale content when DNS lookup fails

Proxies should serve stale content even if DNS lookup to OS is unsuccessful.

For hierarchical caching, if a child proxy matches successfully to a parent proxy and child has requested stale object in its cache, it should not need to make a DNS request for the origin.

It can validate with the parent and return the object as expected.

Without hierarchical caching, if stale content is being requested from a proxy and the DNS lookup to the OS is unsuccessful, it should also serve the stale object

this behavior is similar to when the origin server is down and the proxy serves stale content with a warning header

These changes still respect the max_stale_age configurations and do not serve stale content pass the expiration date.
Autest tests the following using PUSH command to store stale content in proxy's cache:

With child mapping to an origin server without a DNS entry and child matching successfully to parent, a request from child after content becomes stale returns 200

After max_stale_age has passed, request from child returns 500 as stale content is no longer returnable

Without hierarchical caching and proxy mapping to an origin server without a DNS entry, a request from proxy after content becomes stale returns 200

After max_stale_age has passed, request from proxy returns 500 as stale content is no longer returnable

I forget now, did you say Apple is already running this on production proxies?

We discussed this in today's meeting, it turns out microdns is trickier to use than I thought, and not well-documented, so approving this as is.

Cherry-picked to v9.2.x


Amount of active users, events and proposals on the Site

Just a full width block with a text, something like:
Right now we have 22 events, 367 proposals and 15337 users at Speakerfight.
Thats good to show the people that there is a lot of people using it.

I will take that.

PR: #116


Fixed the typo

At Project requirements there is a typo. It is written "includong" it should have been "including"
Closes #724

Thank you!


Missing apply_bpe.py

Hi
I am trying to implement your code following the instruction you noted in README.md.
I found that "apply_bpe.py" which is required in executing "process.sh" does not exist in the repository.
Could you please provide more information about the subword tokenization method?
Additionally, I cannot find the sentencepiece model or BPE model that is suited to the pre-trained model you have released. Is there any released file that I have not found yet?
Thank you

Additionally, could you please provide more instructions about the "TOOLS_DIR=path_to_tools" that is included in inference bash files? What files should we compose in the "path_to_tools" directory for implementing these?

@hyeonseokk Hi, sorry for late reply.
1 apply_bpe.py is the standard script of the toolkit subword-nmt, which can be accessed at https://github.com/rsennrich/subword-nmt.
2 The bpe codes can be found in the directory nmt_dicts_{src_lang}_{tgt_lang}
3 "TOOLS_DIR" is the directory which stores your toolkit, you can push any scripts you need into the tool directory.
Wish I have answered your questions. Feel free to contact us if you have any other questions.

@ZhenYangIACAS Thank you for your reply. I will try it following your instruction.


Migrate existing navbar and sidebar to vuetify

Proposed Changes
This PR migrates the existing navbar and sidebar to Vuetify. https://github.com/falko-org/Falko/issues/382
Type of Changes

[ ] Bugfix
[x] New Feature

Checklist

[ ] This Pull Request has a significant name.
[ ] The build is okay (tests, code climate).
[x] This Pull Request mentions a related issue.

@gabrielademoraes Could you provide some screenshots?

Of course! I have updated the PR.


Indent of copy buttons word wraps passwords on mobile

I'm going to try to fix this.

I give up. I tried everything I know to get rid of padding & margin on  elements using CSS and nothing moves that copy button to the left. I know passwordList in the view should do something about this but I remain baffled how to use those files to control the HTML. Someone else will have to fix this.

I updated the code. The list items are now buttons with visible feedback when clicked. @podfeet  Please test issue-69 branch before I merge it.

Hmmm. I don't see an issue-69 branch...

Looking forward to seeing Helma's change.
@podfeet - the view class "injects" the html

Yeah, I found the <li> element in the view class but I absolutely do not follow how anything around it works. It's hard to even articulate how little I understand looking at this file.

@podfeet sorry, forgot to push. It's there now.

@hepabolu  - I like it. Works great.

Also @hepabolu - for these "one off" html elements like the password list and text area - where do you draw the line on what, if anything, should go into the cascading style sheet vs just having the attributes in the html source? In these cases are there benefits to one over the other?
When it comes to the web development side of things, I am definitely in the "knows enough to be dangerous" camp 😉

This fixes the indent issue. I wish I knew why this fixed it though and removing padding (or margin) on the ul did not. ¯_(ツ)_/¯
I can tell that the entire line gets selected when I tap but in a way that's more confusing. I still don't know that I've copied anything so now I think all I've done is select the entire line.
Can't we just do something simple like have the copy button turn into the solid black version of the icon when pressed?

I just tried adding a .removeClass then an .addClass to the button when pushed to change it to a checkmark. Not sure why the removeClass doesn't actually remove the clipboard icon but it's still not the worst solution.
     for (const pwdIndex in passAndStats.passwords) {
        const btn = $(`#copyclip_${pwdIndex}`);
        if (btn && (btn.length > 0)) {
          btn[0].addEventListener('click', async ()=>{
            // eslint-disable-next-line max-len
            await navigator.clipboard.writeText(passAndStats.passwords[pwdIndex]);
            btn.removeClass('bi-clipboard');
            btn.addClass('bi-check');
          });
        }
      }

Not sure about that. or at least "as written". Perhaps if we changed the icon for only the most recently copied thing? So in you case, if I have 3 passwords and I copy each one, they would all have changed to check marks. Do we want that or as I copy one password, the icon for all others goes back to a checkbox?

or making the content of each list item being a button with the text of the password?? 🤷♂
Trying this now.....

@helma - with the proposed change, I think you inadvertently broke the unregistering of events in fn __resetPasswordUI()
No more buttons.
    // Clear out existing events.
    const existingListItems = this.#passwordList.find('button');
    for (const item of existingListItems) {
      if (item.hasOwnProperty('removeEventListener')) {
        item.removeEventListener('click');
      }
    }

Well this took WAY longer than I anticipated !!
But this is basically @podfeet 's idea, but the check mark is only on the last selected/copied item.
I also switched from the "clipboard" (paste) icon to a real "copy" icon.
** Also - @podfeet , you seem to have been putting the icon (class) on the wrong object. You were manipulating the hyperlink ( "" ), which is named "btn" though it is no longer a button, as opposed to the "" object.
      // Add event handlers for the copy buttons
      // eslint-disable-next-line guard-for-in
      for (const pwdIndex in passAndStats.passwords) {
        const btn = $(`#copyclip_${pwdIndex}`);
        if (btn && (btn.length > 0)) {
          btn[0].addEventListener('click', async ()=>{
            // eslint-disable-next-line max-len
            await navigator.clipboard.writeText(passAndStats.passwords[pwdIndex]);

            // Manage icons
            // Get the <i> elements for the passwords.
            const existingListItems = this.#passwordList.find(`i`);
            for (const item of existingListItems) {
              // Determine the correct icon for the password list items.
              const desiredClass  = (pwdIndex !== item.id) ? 'bi-copy' : 'bi-check';
              const obsoleteClass = (pwdIndex !== item.id) ? 'bi-check' : 'bi-copy';

              // Get an array of classes for the list item.
              let classList = Object.values(item.classList);

              // Remove the "obsolete" class if necessary.
              if (classList.includes(obsoleteClass)) {
                classList =classList.filter((removeClass) => {
                  return removeClass !== obsoleteClass;
                });
              }

              // Appemd the desired class if necessary
              if (!classList.includes(desiredClass)) {
                classList.push(desiredClass);
              }

              // Update the class.
              item.className = classList.join(' ');
            }
          });
        }
      }

I'm not seeing any changes to the copy buttons when clicked in my latest pull from Origin of #69. (and yes I restarted npm)

I don't follow how you did it @pricemi115 but I like it!

Happy to chat and take this as a learning opportunity.....for both of us...
As they say, "the best way to learn is to teach" 🤣

@helma - with the proposed change, I think you inadvertently broke the unregistering of events in fn __resetPasswordUI() No more buttons.
    // Clear out existing events.
    const existingListItems = this.#passwordList.find('button');
    for (const item of existingListItems) {
      if (item.hasOwnProperty('removeEventListener')) {
        item.removeEventListener('click');
      }
    }

Ouch.  Sorry

Also @hepabolu - for these "one off" html elements like the password list and text area - where do you draw the line on what, if anything, should go into the cascading style sheet vs just having the attributes in the html source? In these cases are there benefits to one over the other?
When it comes to the web development side of things, I am definitely in the "knows enough to be dangerous" camp 😉

I just don’t like style attributes because multiple properties are harder to read and it’s an extra location where you look for css.

@pricemi115 and I went through his solution and I love it!

@helma - with the proposed change, I think you inadvertently broke the unregistering of events in fn __resetPasswordUI() No more buttons.

// Clear out existing events.

const existingListItems = this.#passwordList.find('button');

for (const item of existingListItems) {

  if (item.hasOwnProperty('removeEventListener')) {

    item.removeEventListener('click');

  }

}

Ouch.  Sorry

No worries. That's why we do (or should do) peer reviews 😉

Should the automated tests have caught this? I must confess I've forgotten to run them the last few times I've messed in the JavaScript.

Should the automated tests have caught this? I must confess I've forgotten to run them the last few times I've messed in the JavaScript.

If I had tests for this part they would. So far my attempts to add jest tests for these files have failed and I forgot to test the preset buttons by hand.

And even if there were tests for the view class, catching events being left behind would be a challenge...I suspect

I added an adjusted version of @pricemi115 's snippet. And pushed it to the issue-69 branch. Please review @podfeet and @pricemi115. If it works, I can add it to the main branch.

I like the "adjusted version" and it works as expected. One question/comment: Is there any concern or issue with calling removeClass() where the class does not exist? And conversely, calling addClass() where the class already exists?
Also, I am not seeing the check mark as bold or very large on Safari and Firefox on macOS 14.4.

Bart specifically taught us to use removeClass() before adding a class just in case so I don't think there's any problems introduced by using that technique.
The changes look good to me.

I like the "adjusted version" and it works as expected. One question/comment: Is there any concern or issue with calling removeClass() where the class does not exist? And conversely, calling addClass() where the class already exists?

AFAICT it looks like the jQuery functions 'try' to remove/add classes, but the browser makes sure nothing happens when the class  doesn't exist or already exists.

Also, I am not seeing the check mark as bold or very large on Safari and Firefox on macOS 14.4.

I know, I've tried, but the size is fixed to 16x16 and the bold is also not very obvious. I guess that's due to the way the bootstrap icons are set up. Fontawesome offers their icons as a font which can then be resized, bolded and do all kinds of nifty things like overlaying them to create new icons.

Thanx for the follow up and explanation


addition of iommu arguments to kernel boot args

PR superseedes: https://github.com/harvester/harvester-installer/pull/380
It fixes issue: https://github.com/harvester/harvester/issues/2973 by moving the iommu argument definition out of harv-installer and into bootargs.cfg.
This will ensure args are applied during both install and upgrade.

I believe Tobi tested this and determined that if one of the flags doesn't apply, it's simply a no-op.


Deploy

@lonnieezell @MGatner The deployment failed.

Done
=========================================================================
Start time of synchronization  ->  Mon Sep  4 05:25:44 UTC 2023
Warning: Permanently added '***' (ED25519) to the list of known hosts.
***@***: Permission denied (publickey).
rsync: connection unexpectedly closed (0 bytes received so far) [sender]
rsync error: unexplained error (code 255) at io.c(231) [sender=3.2.7]

https://github.com/codeigniter4projects/website/actions/runs/6069476407/job/16463912346

I will look on desktop later.

See also https://github.com/codeigniter4/CodeIgniter4/issues/7464 if you have time.

Ah, I see. Thank you. My SSH key no longer works either so I want to make sure there was no funny business before I regenerate everything from hosting.

It looks like the SSH key had gotten removed at some point. I added it back to the server, installed a missing php-xml extension, and re-ran the action. It says it succeeded but the new website isn't live.

It looks the deploy action did the job:
https://github.com/codeigniter4projects/website/actions/runs/6069476407/job/17164450784

@lonnieezell It seems it restarted php7.4-fpm.
cd $RELEASE_DIR
sudo ln -nsf $RELEASE_DIR/$RELEASE "../current"
sudo service php7.4-fpm reload

https://github.com/codeigniter4projects/website/actions/runs/6069476407/job/17164450784

New site is live! All it needed was to restart php. However, it seems we missed a limit on the "news" section where it pulls from the forums. I'll put together a PR for that now, and will include the fix for the deploy script as well.

See #402

I deployed #402
See #403
Now the site works fine! 🎉🎉🎉


Avoid eval() when reading config file

Using eval() has unintended side-effects, and may be considered a security risk. Instead, do nothing if we expect a string, and use ast.literal_eval() for other types (list, tuple, etc.). Also add tests.
Fixes #3354. That issue also talks about problems with IPython startup configuration, which I will leave for @ccordoba12.

:+1: LGTM

@jitseniesen, your new test is failing in Python 2.7.

@jitseniesen, your new test is failing in Python 2.7.

Just wait a couple of years and we don't have to care anymore :)
I guess I should have tested more carefully with Python 2 and I would have noticed that the ini file is in a different format, but at least it's good that the test caught it. I fixed the issue and I also rebased so that it should merge after the spyderlib rename. Fingers crossed that the tests pass this time.

Thanks @jitseniesen for updating your work :-)
I left some comments related to your rebase, the rest seems right ;-)

I left some comments related to your rebase, the rest seems right ;-)

@ccordoba12 Thanks for having a look. Something went wrong with the rebase which I'm in the process of fixing.

All should be fine now.


Suggestion: Change Modify step in Getting Started to truly Modify

Right now in our Getting Started flow, we go from one example (an S3 bucket) to something entirely different (an EC2 instance running a Python web server). I personally found this confusing to label as "modifying" my program, since I'm actually replacing it altogether. Not only is the EC2 instance example bigger and harder to follow than the nicely simple S3 one we started with, this seems like a missed opportunity to give me a sense of truly updating resources. I would encourage thinking about something else instead, like enabling encryption at rest on our S3 bucket, which is simpler and will give the end user a quicker, more relevant feel for what a typical update entails.

Another reason to go with the S3 example, rather than EC2. I think if you have an old AWS account created prior to 2013 -- like I do -- it still tries to use EC2-Classic by default when deploying into us-east-1, which fails with the default steps specified in our tutorial:
    error: Plan apply failed: Error launching source instance: VPCResourceNotSpecified: The specified instance type can only be used in a VPC. A subnet ID or network interface ID is required to carry out the request.

I might be mistaken, but I think that's what's going on.

For instance, this is a nice modification to the base Python example that creates a KMS key and enables encryption on our S3 bucket:
import pulumi
from pulumi_aws import kms, s3

# Create a KMS Key for S3 server-side encryption;
key = kms.Key('my-key')

# Create an AWS resource (S3 Bucket)
bucket = s3.Bucket('my-bucket',
    server_side_encryption_configuration={
        'rule': {
            'apply_server_side_encryption_by_default': {
                'sse_algorithm': 'aws:kms',
                'kms_master_key_id': key.id
            }
        }
    })

# Export the name of the bucket
pulumi.export('bucket_name',  bucket.id)

It can be done with an in place update, and is pretty easy to follow:
Updating (dev):

     Type                 Name            Status      Info
     pulumi:pulumi:Stack  quickstart-dev
 +   ├─ aws:kms:Key       my-key          created
 ~   └─ aws:s3:Bucket     my-bucket       updated     [diff: ~serverSideEncryptionConfiguration]

Outputs:
    bucket_name: "my-bucket-127bd18"

Resources:
    + 1 created
    ~ 1 updated
    2 changes. 1 unchanged

Duration: 11s

PR for AWS: #1253
I look into doing something similar for Azure and GCP.

[ ] AWS (#1253)
[ ] Azure
[ ] GCP

Fix is available for AWS, but still need to update Azure and GCP. Moving to next milestone.


Added new method to list events of Hawkular, added tests.

Is there an alternative of "sys.maxsize" to get Long.MAX_VALUE?

Coverage increased (+0.3%) to 30.167% when pulling 07265a98361ecab673a07b886d506e9ab9d22ad6 on hhovsepy:hawkular_integration into b180138d1a2a85f6afc7f2b74971b4425d87df04 on RedHatQE:master.

Lint report for commit 07265a98361ecab673a07b886d506e9ab9d22ad6:
mgmtsystem/hawkular.py:

:red_circle: Line 199:9: N803 argument name should be lowercase

Please, rectify these issues :smirk: .
CFME QE Bot

Coverage increased (+0.3%) to 30.167% when pulling c1f7172d18504d59ef2676bacbae52cc381efda5 on hhovsepy:hawkular_integration into b180138d1a2a85f6afc7f2b74971b4425d87df04 on RedHatQE:master.

Lint report for commit c1f7172d18504d59ef2676bacbae52cc381efda5:
Everything seems all right :smile: .
CFME QE Bot

@hhovsepy What is wrong about the sys.maxsize?

@mfalesni  nothing, was asking if there is other more recommended way.

Lint report for commit 14fa1cfcf11271b11c080f38c752c8ec878b7d41:
Everything seems all right :smile: .
CFME QE Bot

Coverage increased (+0.3%) to 30.138% when pulling 14fa1cfcf11271b11c080f38c752c8ec878b7d41 on hhovsepy:hawkular_integration into ab273a3e2c4d148a5409874b44dec035c4d1df90 on RedHatQE:master.


possibility to skip the upload / download if the file exists

hello, can you add the option to skip the upload / download if the file exists

Hey @vince87 , thanks for raising this issue. I'll work on this and update you shortly.

The latest version (2.0.1) fixes this. Test and let me know.


Thread - Thread preview in LHN shows "left the chat" after leaving thread in thread page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.33-3
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause- Internal Team
Slack conversation:
Action Performed:

Go to any chat.
Send a message.
Right click on the message > Leave thread.
Right click on the message > Reply in thread.
Note that LHN shows "No activity".
Return to main chat.
Send another message.
Right click on the message > Reply in thread.
Click 3-dot menu > Leave thread.
Right click on the message > Reply in thread.

Expected Result:
The thread preview in LHN will show "No activity".
Actual Result:
The thread preview in LHN will show "left the chat".
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/8d44c6f2-bbc9-4566-a286-a6822d5483de
View all open jobs on GitHub

We think that this bug might be related to #vip-vsb
CC @quinthar

Closing for now since I can't reproduce. I asked in #QA to reopen the issue if they find consistent reproduction steps.

QA team is able to reproduce it with above steps
https://github.com/Expensify/App/assets/93399543/cd85347c-b44d-4982-b47c-e2e04a047797

Updated the original description to focus on the fact that message without threads have a "Leave Thread" option.

This has been discussed in multiple places already – it's expected behavior – similar to Slack's "do not receive notifications for this message" – it happens only if you're the author of the message.

https://github.com/Expensify/App/pull/27994/files#r1365082102
cc: @srikarparsi

We've come to the conclusion to remove the "Leave thread" option for all comments in the comment's overflow menu but keep the "Join thread". Slack

Proposal
Please re-state the problem that we are trying to solve in this issue.
The "Join Thread" option appears for comments in the context menu.
What is the root cause of that problem?
We explicitly allow these options to appear for the actions, and this change is like an update request:
https://github.com/Expensify/App/blob/caf91aefc70e8bb82813a421097fd37b412bb137/src/pages/home/report/ContextMenu/ContextMenuActions.tsx#L274-L285
What changes do you think we should make in order to solve the problem?
If we want to hide the "Join Thread" options for the "hover" menu completely, we can just remove this option:
https://github.com/Expensify/App/blob/caf91aefc70e8bb82813a421097fd37b412bb137/src/pages/home/report/ContextMenu/ContextMenuActions.tsx#L270-L300
Otherwise, if we want to show it only when there already is a thread, we should early return false in the corresponding shouldShow function if !lodashGet(reportAction, 'childReportID', '0').
What alternative solutions did you explore? (Optional)

@srikarparsi could you please share the context for a better understanding of the expected behavior? Contributors don't have access to the Slack link you provided

Should the comments with deleted replies show "Leave Thread"?

Proposal
Please re-state the problem that we are trying to solve in this issue.

In chat, messages with no thread show "Leave Thread" on the context menu.

What is the root cause of that problem?

There is not dedicated function to check if the message has or had a thread.

What changes do you think we should make in order to solve the problem?

Add a check in ReportActionsUtils to check if a ReportAction has "children",  a thread.

What alternative solutions did you explore? (Optional)

I feel there are too many checks for a thread. There should be only one check for this context menu, if the message has a thread.

Hey @paultsimura, your proposal looks good, we just need to remove the leaveThread option from the overflow menu for all comments. When do you think you can get a PR out by?

Thanks @srikarparsi, I can make it today

Thanks. The PR is ready for review: https://github.com/Expensify/App/pull/35573

I paid out @paultsimura.
@Ollyws could you please accept the Upwork offer and also complete the BZ checklist? Thanks!

Thanks! All paid out 🙌


Blinking on a long press gesture

When I simply tap and hold a touch the application blinks.
Simulator 8.1 but it seems it will occur in all the versions

changing
    self.cell!.drawViewHierarchyInRect(self.cell!.bounds, afterScreenUpdates: true)

to false solved it to me.


Unable to install opa-wasm

Description:
Unable to install opa-wasm following the install instruction. Installed without the cranelift or llvm was fine, however.
Environment: MacOS M1
Steps took
[~]$ pip install opa-wasm[cranelift]
zsh: no matches found: opa-wasm[cranelift]
[~]$ pip install opa-wasm[llvm]
zsh: no matches found: opa-wasm[llvm]
[~]$ pip install opa-wasm
Collecting opa-wasm
  Using cached opa_wasm-0.3.2-py3-none-any.whl (5.1 kB)
Collecting wasmer<2.0.0,>=1.0.0
  Using cached wasmer-1.1.0-py3-none-any.whl (1.6 kB)
Installing collected packages: wasmer, opa-wasm
Successfully installed opa-wasm-0.3.2 wasmer-1.1.0

Versions
[~]$ pip --version
pip 23.0.1 from /opt/homebrew/lib/python3.10/site-packages/pip (python 3.10)

Question

Did I use the incorrect command ?

This appears to be an issue with zsh and the square bracket syntax.
Try using pip install "opa-wasm[cranelift]" (with quotation marks)? If that works then the README.md should be updated to reflect this, since from my tests it appears as if both bash and zsh are fine with quotation marks.

This appears to be an issue with zsh and the square bracket syntax.
Try using pip install "opa-wasm[cranelift]" (with quotation marks)? If that works then the README.md should be updated to reflect this, since from my tests it appears as if both bash and zsh are fine with quotation marks.

That was it, thank you. I am creating a PR to update the instruction.
https://github.com/a2d24/python-opa-wasm/pull/8


Added digitalocean terraform sample

Signed-off-by: Graham Weldon<EMAIL_ADDRESS>

Hello @predominant , as someone who is very interested in seeing this PR be merged, I think it would be nice if you could follow the example set by the existing AWS Terraform code and be a bit more descriptive of your variables, as well as declaring type (with defaults where merited). Just my .02 (@chefsalim @eeyun)

Done @danielcbright

Looks good @predominant ... would you mind also squashing your commits?

@chefsalim Squashed


added passthrough options

Added passthrough options for a keyfile, as requested in https://github.com/meztez/plumberDeploy/issues/12
Description
Now you can specify a keyfile for any operation that requires ssh.  Related to https://github.com/sckott/analogsea/issues/155
Related Issue
https://github.com/sckott/analogsea/issues/155

thanks @muschellij2
sadly i still don't have tests for most things here. have you tested the added parameters yourself?

@Adamoso - I made this to try to address your issue at https://github.com/rstudio/plumber/issues/524.  Would you be able to test?

Hello @muschellij2!
Thank you for addressing this issue. I will test it and share my results here.

Any thoughts on this?  I think travis fails were due to DO tokens.

@muschellij2 are you asking me?  or are you waiting on response from @Adamoso ?

Apologies, asking from @Adamoso.  Also, unsure if the failures are true or false positives, which I think @sckott can answer.  Also, just seeing if you need additional tests for this.  I don't pass in keyfiles or other params to use this, but I think it could be useful if you do.

@muschellij2 sorry for taking so long! I was finally able to test most of the functionalities with two different keys. Everything seems to function properly. I can't however find any reason the tests are failing as when I was using the modified functions without specifying new parameters it seemed to work just the same as before.

tests failing b/c token not available in PR's
having a look

looks good to me


Add fallback to SDK for .NET Framework projects

If we can't find a legacy .NET Framework MSBuild toolset, fallback to locating a .NET Core SDK toolset since it can also build legacy .NET Framework projects.

This is OBE now that I know the .NET Core SDK can't actually build all .NET Framework projects - you need the normal VS or VS Build Tools toolchain for .NET Framework projects, even if they're using the SDK. See #23.


Breaks app

Breaks the following app: https://github.com/chenxiaolong/BCR/issues/586#event-13933101885

Device:  Samsung Fold 3
OS:  A14, OneUi6
Version of Magisk 27000

Nothing is unmounted unless the target app is in the deny list. Make sure that the app is not in the deny list.

Alright, thanks. Its not in the deny list. Maybe the issue lies with tricky_store then.

Just for info if someone needs the solution in the future, install BCR with module. Right after, extract the apk from the module zip file and update the BCR application with the extracted apk file.

Could you confirm if removing everything from the deny list and disabling TS fixes the problem? In that scenario the app should be visible to launchers and all other apps.

Could you confirm if removing everything from the deny list and disabling TS fixes the problem? In that scenario the app should be visible to launchers and all other apps.

No need to remove the list in deny list. Simly Disabling TS fixed the issue. But o found a real solution to the problem mentioned above.


Feat: Add opportunity to change CupertinoTextField suffix alignment

Feat: #152482
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on [Discord].

@Piinks, Can u check please

Hello @Piinks, OK.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 378

Questions ?
Please refer to the Backport tool documentation


Instructions for How the CTI Tag Generator Works

How It Works

Civic Tech Index - How it Works

Civic Tech Index - How it Works


feat(category_theory/limits): limits and colimits

Limits, products, equalizers, pullbacks, binary_products, terminal objects, and all the dual notions, too.
The basic design centres around:
/--
A `c : cone F` is:
* an object `c.X` and
* a natural transformation `c.π : c.X ⟹ F` from the constant `c.X` functor to `F`.
-/
structure cone (F : J ⥤ C) :=
(X : C)
(π : (const J C).obj X ⟹ F)

structure is_limit (t : cone F) :=
(lift : ∀ (s : cone F), s.X ⟶ t.X)
(fac'  : ∀ (s : cone F) (j : J), (lift s ≫ t.π.app j) = s.π.app j . obviously)
(uniq' : ∀ (s : cone F) (m : s.X ⟶ t.X) (w : ∀ j : J, (m ≫ t.π.app j) = s.π.app j),
  m = lift s . obviously)

and a typeclass bundling these together:
class has_limit {J : Type v} [small_category J] (F : J ⥤ C) :=
(cone : cone F)
(is_limit : is_limit cone)

Theorems/constructions included so far:

Type u has all limits and colimits.
products, equalizers, pullbacks from limits
limits from products and equalizers
binary_products and terminal objects from products
if has_limits_of_shape J C, then has_limits_of_shape J (K \func C), and the evaluation function at k preserves limits
F.cones is the category C^op \func Type, giving the collection of cones with a given cone point. We show has_limit F iff representable F.cones

403 - 188 = 215 uses of simp
106 - 29 = 77 uses of obviously
70 - 2 = 68 uses of tidy

403 - 188 = 215 uses of simp
106 - 29 = 77 uses of obviously
70 - 2 = 68 uses of tidy

I think you're short-changing me there, @kckennylau. When I add
@[obviously] meta def verbose_obviously (prev : tactic unit) : tactic unit :=
do
  tactic.trace "calling obviously",
  prev

It tells me that obviously is called 104 times in the limit/ directory. (Presumably it's all the auto_params that aren't even visible.) I wish I could get it much higher --- there are too many boring proofs in this PR that should be omitted. Keeley's work on rewrite_search will be a big improvement, and if that ever lands in mathlib we'll be able to automate even more.

Hm, now we have some conflicts @semorrison can you rebase it?

Hmm, crap, I think I did that wrong, and will have to open a new PR.


algoliasearch.exceptions.AlgoliaUnreachableHostException: Unreachable hosts

this my code:
from nintendeals import noa

def getUSGamesBrief():
    index = 1
    for game in noa.list_switch_games():
        print(index, "  ", game.title)
        index += 1

if __name__ == '__main__':
    getUSGamesBrief()

and it throw an exception when i run it:
/usr/local/bin/python3.9 /Users/mac/PycharmProjects/switch/venv/gamesUS.py
Traceback (most recent call last):
  File "/Users/mac/PycharmProjects/switch/venv/gamesUS.py", line 12, in <module>
    getUSGamesBrief()
  File "/Users/mac/PycharmProjects/switch/venv/gamesUS.py", line 7, in getUSGamesBrief
    for game in noa.list_switch_games():
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/nintendeals/noa/listing.py", line 52, in list_switch_games
    yield from list_games(Platforms.NINTENDO_SWITCH)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/nintendeals/noa/listing.py", line 10, in list_games
    for data in algolia.search_by_platform(platform):
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/nintendeals/noa/api/algolia.py", line 63, in search_by_platform
    games = _search_index(query, **options)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/nintendeals/noa/api/algolia.py", line 34, in _search_index
    response = INDEX.search(query, request_options=options)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/algoliasearch/search_index.py", line 284, in search
    return self._transporter.read(
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/algoliasearch/http/transporter.py", line 47, in read
    return self.request(verb, hosts, path, data, request_options, timeout)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/algoliasearch/http/transporter.py", line 72, in request
    return self.retry(hosts, request, relative_url)
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/algoliasearch/http/transporter.py", line 94, in retry
    raise AlgoliaUnreachableHostException("Unreachable hosts")
algoliasearch.exceptions.AlgoliaUnreachableHostException: Unreachable hosts

Do you know why this happened？

Hi @Peter-Huang0623 !
All tests for nintendeals seem to be working corretly: https://travis-ci.com/fedecalendino/nintendeals
So I'm guessing there is an issue with your connection.
I also tried your code snippet and its working as intended in my laptop.
Cheers, Fede.


tblite-python is not importable / pkgconfig for mtclib, s-dftd3, dftd4

Describe the bug
tblite is unusable in python at the moment, failing with
Traceback (most recent call last):
  File "/nix/store/pzbm5ci2h0qlnrzkqxhdm9f0vwr5kal6-python3.10-tblite-0.3.0/lib/python3.10/site-packages/tblite/library.py", line 28, in <module>
    from ._libtblite import ffi, lib
ImportError: /nix/store/pzbm5ci2h0qlnrzkqxhdm9f0vwr5kal6-python3.10-tblite-0.3.0/lib/python3.10/site-packages/tblite/_libtblite.cpython-310-x86_64-linux-gnu.so: undefined symbol: __dftd3_model_MOD___vtab_dftd3_model_D3_model

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/nix/store/pzbm5ci2h0qlnrzkqxhdm9f0vwr5kal6-python3.10-tblite-0.3.0/lib/python3.10/site-packages/tblite/interface.py", line 26, in <module>
    from . import library
  File "/nix/store/pzbm5ci2h0qlnrzkqxhdm9f0vwr5kal6-python3.10-tblite-0.3.0/lib/python3.10/site-packages/tblite/library.py", line 30, in <module>
    raise ImportError("tblite C extension unimportable, cannot use C-API")
ImportError: tblite C extension unimportable, cannot use C-API

I think this is due to incorrect pkgconfig setups - look at
https://github.com/NixOS/nixpkgs/blob/e608c90a1cf381dde6ac9e0f085337150f2af3e2/pkgs/development/libraries/toml-f/default.nix#L26
which is correct and outputs
prefix=/nix/store/8irr24ds1c3r08wclk11gpxp3c7f0p7w-toml-f-0.3.1
libdir=${prefix}/lib
includedir=${prefix}/include

Name: toml-f
Description: A TOML parser implementation for data serialization and deserialization in Fortran
Version: 0.3.1
Libs: -L${libdir} -ltoml-f
Cflags: -I${includedir} -I${includedir}/toml-f/GNU-12.2.0

vs
https://github.com/NixOS/nixpkgs/blob/e608c90a1cf381dde6ac9e0f085337150f2af3e2/pkgs/development/libraries/science/chemistry/mctc-lib/default.nix#L27
which is missing the trailing /
Which results in a pkgconfig of
prefix=
libdir=${prefix}//lib
includedir=${prefix}//include

Name: mctc-lib
Description: Modular computation tool chain
Requires:
Version: 0.3.1
Libs: -L${libdir} -lmctc-lib
Cflags: -I${includedir} -I${includedir}/mctc-lib/GNU-12.2.0

Steps To Reproduce
Steps to reproduce the behavior:

Create a nix environment with python3Packages.tblite
In the python interpreter, run import tblite.interface as tb
See failure

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers
@sheepforce
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
output here

i'm having the exact same issue as well :'(


Fix cmdlet not recognized

Command Update-Order does not exist. Replaced with Update-ACMEOrder.

Thank you for pointing the flaw out and fixing it :)


Add Wuchang Shouyi University

@DanielSimgame The domain wsyu.cn that you are submitting does not belong to a university, nor to a college. I have to close the request without merging. Please ensure that you are familiar with this repository rules located at the bottom of this page before submitting a new request.

@DanielSimgame The domain wsyu.cn that you are submitting does not belong to a university, nor to a college. I have to close the request without merging. Please ensure that you are familiar with this repository rules located at the bottom of this page before submitting a new request.

Sorry for that mistake, the right domain is "wsyu.edu.cn", I will correct that.


support DHCP for network config of controlplane and workers

When using DHCP instead of IPAM the Hostname of the VM is not set by DHCP. In these case we need to inject the desired hostname through cloud-init for both control-plane and worker nodes.
A script running on the worker can check hostname and set it based on temporary file if needed (silimiar to VMware implementation).
As alternative we could set /etc/hostname directly, but we must evaluate if this impacts other ways of configuring the hostname.
A suitable cloud-init extra argument could look like this:
`write_files:

content: |
myhostname
path: /tmp/hostname`

Created https://jira.nutanix.com/browse/KRBN-6008 to track.

Related RH slack discussion: https://coreos.slack.com/archives/C0211848DBN/p1666968673015319


🧪 Add vitest

Hi @peterbud
We can continue our testing related conversation here.
Is this change necessary?

I was not sure, to be honest. The reason of those was that I have seen that null values in the slots are converted somewhere in the rendering to <!----> - my guess that it is an issue of the JSX transpiling, but have not tested.
Example:
You hvae a button, and the default slot is 'Click me'
If you have null there as in the original, the button is rendered as (note the two <!----> for each null slot ):
<button style="...">
  <!---->Click me
  <!---->
</button>

if you use empty strings instead of null, these placeholders are not rendered.
<button style="...">Click me</button>

It's really a cosmetic stuff, and I think the original version with the null is the best, just would be nice to render them as empty, instead of such placeholders.
I'll revert back to null.

Hi @peterbud
I'm ready to learn vitest now. Do you have any fine resources?
Moreover, it will be great if you can suggest a scalable folder structure for Anu's tests.
Thanks.

Ohk, I will start with docs then.
Mostly, I noticed people have separate test folder, 1st option.

Btw, i must ask as we have component library what's suitable?

Another option is, should we have tests inside each package instead of single test folder in root 🤔

I have also found a good example for component testing for element-plus:
https://github.com/element-plus/element-plus/blob/dev/packages/components/button/__tests__/button.test.tsx
The keep the tests next to each component, and I think it's a good example of what they are testing in the components

Hmm, That's another approach.
After checking out some other libs I think we should place tests dir in the root, outside of packages, your 1st option.
Can you give us a boost start?

Sure, I'll rebase the vitest branch to the latest main branch, and I'll open a PR, and we can continue the work there.


Add neg math spacing.

This adds a negative -1/6em math spacing option for parity with LaTeX's \!. This can be useful in various situations, especially to manually fix inconsistencies.

Wouldn't it make more sense to have it be thin.neg?

For complete parity with LaTeX you should also add negative variants of med and thick, and if you do that you might as well include quad and wide too. LaTeX’s enspace is also missing.

For complete parity with LaTeX you should also add negative variants of med and thick

Not sure what you mean. Isn't this the only named one?

\negmedspace and \negthickspace both exist, they just don't have a shorthand.

Wouldn't it make more sense to have it be thin.neg?

Not possible currently since it's not a symbol, but rather just a #let thin = h(-1em/6) :/
But in principle I agree with you.

I also agree that *.neg would be a better option. Is there a reason why in math mode h(...) instead of the symbols is used?

Symbols are strings. I'm not sure there are Unicode codepoints for all these space widths.

You are right, I found the following Unicode characters on the whitespace wiki page

thin, 1/6em: THIN SPACE U+2009 (1/5em or 1/6em) or SIX-PER-EM-SPACE U+2006
med, 2/9em=1/3em: MEDIUM MATHEMATICAL SPACE U+205F (THREE-PER-EM SPACE U+2004 exists as well)
thick, 5/18em: ?
quad, 1em: EM SPACE U+2003 or EM QUAD U+2001
wide, 2em: ?

Also, sorry if I started this the wrong way, I probably should have explained more in the original post.
This was more meant as a discussion starter because I found implementing it much easier than explaining what I mean.

Since we can't have the nice design (thin.neg) with our current infrastructure, I'd be inclined to close this for now and maybe revisit the topic in the future.


Dynamic call to static method truncate()

Larastan Version: 0.4.3
--level used: max

Description
I'm getting the following:

Dynamic call to static method Illuminate\Database\Eloquent\Builder::truncate().

When I use
DB::table('users')->truncate();

there's no error.
Laravel code where the issue was found
Model::query()->truncate();

I can't reproduce this. I tested with User::query()->truncate(); in our test cases.
Can you give more information or make a PR with failing test case?

Alright, so here's what I have:
Test.php
<?php

declare(strict_types=1);

namespace App\Models;

use Illuminate\Database\Eloquent\Model;

class Test extends Model
{

}

and the call in a dummy method in a controller:
public function test()
{
    Test::query()->truncate();
}

Here's my phpstan.neon.dist:
includes:
    - vendor/phpstan/phpstan-deprecation-rules/rules.neon
    - vendor/phpstan/phpstan-strict-rules/rules.neon
    - vendor/phpstan/phpstan-mockery/extension.neon
    - vendor/phpstan/phpstan-phpunit/extension.neon
    - vendor/phpstan/phpstan-phpunit/rules.neon
    - vendor/nunomaduro/larastan/extension.neon
parameters:
    level: max
    paths:
        - app
        - config
        - database
        - routes
        - tests
    ignoreErrors:
        - '~UuidInterface~'

        # Ignore this since it would clash with our style rules
        - '~has no typehint specified~'

The issue is here. It's hardcoded to return true Since all query builder methods can also be called statically, this was an easy solution.
So, this needs to be changed, somehow.

Model::toBase() may cause this error too.

Just wanted to say that I am still experiencing this with 0.5.2.


RC 3.0 on Windows

I have tried to run DDLFS on windows but it seems like nothing really happens.
After a successful connection to the DB, the connection is closed and no filter is created and no DDL files are vilible.
I ran the DDLFS with debug and here is what I got on my screen:

The text version of the screenshot (user, schema, password and service_name are redacted)
PS C:\Users\JacekGebal> ddlfs -o username=*****,password=*****,database=localhost:1521/*****,loglevel=DEBUG,schemas=***** ./dbobjects -f

C:\Users\JacekGebal>start /B cmd /c ddlfs-fg.bat -o username=*****,password=*****,database=localhost:1521/*****,loglevel=DEBUG,schemas=***** ./dbobjects -f
PS C:\Users\JacekGebal> 2024-12-03 11:10:41 INFO  DDL Filesystem v3.0-RC1 for Oracle Database, FUSE v2.7
2024-12-03 11:10:41 DEBUG Parameters:
2024-12-03 11:10:41 DEBUG .. username : [*****]
2024-12-03 11:10:41 DEBUG .. password : [****]
2024-12-03 11:10:41 DEBUG .. database : [localhost:1521/*****]
2024-12-03 11:10:41 DEBUG .. loglevel : [DEBUG]
2024-12-03 11:10:41 DEBUG .. schemas  : [*****]
2024-12-03 11:10:41 DEBUG .. userrole : []
2024-12-03 11:10:41 DEBUG .. temppath : [C:\Users\JACEKG~1\AppData\Local\Temp]
2024-12-03 11:10:41 DEBUG .. filesize : [0]
2024-12-03 11:10:41 DEBUG .. keepcache: [0]
2024-12-03 11:10:41 DEBUG .. pdb      : []
2024-12-03 11:10:41 DEBUG .. dbro     : [-1]
2024-12-03 11:10:41 DEBUG .
2024-12-03 11:10:41 DEBUG Connecting to Oracle Database (localhost:1521/*****)
2024-12-03 11:10:41 DEBUG .. allocating handles.
2024-12-03 11:10:41 DEBUG .. attaching server process.
2024-12-03 11:10:41 DEBUG .. setting session attributes (username=*****).
2024-12-03 11:10:41 DEBUG .. starting [default] database session.
2024-12-03 11:10:42 DEBUG .. registering database session.
2024-12-03 11:10:42 DEBUG .. determining database version.
2024-12-03 11:10:42 DEBUG .. connected to server version [Oracle Database 19c] [1900]
2024-12-03 11:10:42 INFO  .. connected to database server.
2024-12-03 11:10:42 DEBUG is_sysdba=[0]
2024-12-03 11:10:42 DEBUG assuming database opened as read/write because this user has no (sys)dba privileges (needed to obtain this info).
2024-12-03 11:10:42 DEBUG dbro=[0]
2024-12-03 11:10:42 DEBUG tfs_rmdir, folder [C:\Users\JACEKG~1\AppData\Local\Temp\ddlfs_localhost_1521_*****.tns.*****.*****] does not exist (which is OK).
2024-12-03 11:10:42 DEBUG tfs_mkdir - created temporary directory: [C:\Users\JACEKG~1\AppData\Local\Temp\ddlfs_localhost_1521_*****.tns.*****.*****]
2024-12-03 11:10:42 DEBUG
2024-12-03 11:10:42 DEBUG -> event-loop <-

2024-12-03 11:10:42 DEBUG
2024-12-03 11:10:42 DEBUG -> umount <-
2024-12-03 11:10:42 INFO  Disconnecting from Oracle Database.
2024-12-03 11:10:42 DEBUG .. freeing handles
2024-12-03 11:10:42 DEBUG .. terminating OCI.
2024-12-03 11:10:42 INFO  .. disconnected.
2024-12-03 11:10:42 INFO  Cache directory [C:\Users\JACEKG~1\AppData\Local\Temp\ddlfs_localhost_1521_*****.tns.*****.*****] successfully removed (config keepcache=0)

PS C:\Users\JacekGebal>

I am connected with an account that has a DBA role (not SYSDBA) bit I am not sure if this is the issue.
Am I doing something wrong?

It seems that it doesn't like relative mountpoint.
Replace ./dbobjects with absolute path to an empty folder, or choose any unused letter, e.g. X:
btw, for easier debugging, you can run "ddlfs-fg" command instead of "ddlfs".

So I tried mounting the DDLFS as a sub-folder on my C:\dev\my_project\db_objects where directory db_objects, does not exist. This still did not work.
I was able to mount it as a new drive letter X:
Is it a bug or a feature?
I have succesfuly started ddlfs-fg that is now running as foreground process on my Windows.
I have killed the terminal that was running the foreground process but the ddlfs seems to be still running and I can still access the X: drive and its contents.
How do I stop, check status, restart a running ddlfs?

Empty directory must already exist before mount ddlfs. Create empty directory C:\dev\my_project\db_objects and try again. db_objects will be replaced with junction on mount and back to folder on unmount. But it must exist as an empty folder to begin with.
I could call "mkdir()" before mount, but I believe that would not be expected behaviour, so maybe simply a better error message would be a good idea in such case.
Execute ddlfs-umount X: to umount (regardless of whether you run in foreground or background).


Undo database write if any step failed in createTable()

Undo any steps already taken by createTable() if it failed halfway through.
From inline notes:

// NOTE: potentially undo stuff that's done at this point (DynamicSchema: L212)

Reverse steps in place for both createTable() and save().


Test for code formatting as PR

This is to remove the use of reviewdog in current workflow that could easily overflows the comments in a PR.

Codecov Report

Merging #361 (8808eda) into master (c9206a4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #361   +/-   ##
=======================================
  Coverage   52.84%   52.84%
=======================================
  Files         127      127
  Lines       19179    19179
=======================================
  Hits        10135    10135
  Misses       9044     9044

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

I also changed the docker actions so that the arm64 arch is only built in the weekly schedule.


Refactored event registration api and added email functionality.

email logs in the terminal on local

Pleasure.

On 16 Sep 2024, at 18:16, Bhuvnesh @.***> wrote:
Thanks for reviewing @ankanchanda https://github.com/ankanchanda @engagepy https://github.com/engagepy 👍
—
Reply to this email directly, view it on GitHub https://github.com/djangoindia/djangoindia.org/pull/120#issuecomment-2352820627, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKG4K7ZW4CQULWHNDRAM2JDZW3HIXAVCNFSM6AAAAABNKYYKG6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJSHAZDANRSG4.
You are receiving this because you were mentioned.


[UI] [API Key] API Key password shows up in Edit Screen

Server Version: v0.11.1
When creating the API key, there is a warning that this is the only time the API password will be shown. But if I click on the Edit button, the Edit API Key screen shows the password.

The API shouldn't be providing the secret back to me except on create..

Addressed and fixed in rancher/cattle#618

Verified Fixed in v0.29.0-rc2


Rename get_handler to mut_handler and add method for immutable access to handler

Thanks. One nit, I think the mut should be postfix to follow the standard library. If you change it to handler_mut, I am happy to merge that.


418, I'm a teapot not supported

org.jetbrains.ktor.http.HttpStatusCode does not contain the 418 I'm a teapot status code.

To be honest, this is not part of the specification, and recently there has been concerns about people actually using this code for the wrong reason. It was merely a joke when it came out. I'd be hesitant in adding it. @orangy ?

418 status code is described in RFC 2324 (Hyper Text Coffee Pot Control Protocol) and it's April Fools joke.

Also, you can add it as an extension property
val HttpStatusCode.Companion.IAmATeaPot get() = HttpStatusCode(418, "I'm a tea pot")

somebody should publish the lib with such extension!

I dont think one line of code is worth publishing a lib


Unification of emit

I was lurking around codebase trying to move something towards #201
What I found is a bit puzzling me why we have 4 different abstractions
public interface ITopLevelNode
{
    void EmitTo(TranslationUnitContext context);
}

interface IConstant
{
    void EmitTo(IDeclarationScope scope);
}

internal interface IBlockItem
{
    void EmitTo(IDeclarationScope scope);
}

internal interface IExpression
{
    void EmitTo(IDeclarationScope scope);
}

That does not sounds right for me. so what do you guys think about having
class Emitter
{
   public void EmitDefinition(ITopLevelNode node, TranslationUnitContext context);
   public void EmitBlockItem(IBlockItem blockItem, IDeclarationScope scope);
   public void EmitExpression(IExpression expression, IDeclarationScope scope);
}

I deliberately remove EmitConstant since emitter can know how to emit constant expression, and there no point to have that as separate method. Internally maybe.
Also I do not see a value in the IConstant over simple IExpression. What special treatment constants require? We have add property bool IExpression.IsConstExpression and it would be the same.
Also why we have special ITopLevelNode and not just special IBlockItem which represent top-level definition. Not sure about that one, but even if this definitions require some additional context, that can be handled by giving IDeclarationScope access to TranslationUnitContext within which emit happens.

I can't imagine the implementation of such Emitter, what would it be? A huge switch on types? I'd say this is a step back from the current design.

Also I do not see a value in the IConstant over simple IExpression.

Currently, expressions and constants have no relation to each other. Constant is a more low-level item than an expression. You can check usages of this method to see the context.

Also why we have special ITopLevelNode and not just special IBlockItem which represent top-level definition.

Because there's technically no block on the top level. Though this one may be improved I think.

A huge switch on types?

Yes. So far, that's only option that I see. I dislike it with a passion too, but

I'd say this is a step back from the current design.

Consider that right now IR nodes know about Cecil, and process of emitting is spread across whole IR tree which is bad. I expect that a lot of IR nodes will be never emitted, like if/for/while/switch and instead of them, other nodes which is smaller in nature to be used as basis for emit.
Also consider how we will have separate processes for IR transformations? I worry a lot that we will mix concepts from different phases if everything would be in single class.

Constant is a more low-level item than an expression.

I got your point. Agree. I would say from quick glance, too many stuff knows about how to emit itself. I would like to have IR more of internal representation, and not internal logic.

Though this one may be improved I think.

By itself it's fine, I probably would like to have ITopLevelNode to be renamed to IDefinition or something closer to that that. Just name changes IMO helps. But whole changes in that areas, depends on how our discussion would be flowing.

A huge switch on types?

I think by default I think that it would be some method which visit base interfaces within each would be switches. so not that much problematic, but some areas probably would have a larger then usual switch.

I would say from quick glance, too many stuff knows about how to emit itself

Because this wasn't recognized as a problem when the stuff was designed. And, to be honest, I'm still not sure this is a current problem; though that's possibly because I can't see the limits of the current design well, yet.

Now I think I have rationale why I want to have Emitter class which represent process of emitting.
Currently we have 2 processes called Lower and Emit and will have more in the future. Logic of these processes spread thin across different classes. So rationalize what’s going on become harder and harder. Common way to have logic for single process within single file is to have construct like this https://github.com/dotnet/roslyn/blob/main/src/Compilers/Core/Portable/Symbols/SymbolVisitor.cs
Whether we use exact pattern or not for emit this is subject to discussion, but right now I have more incentive push for combining logic of single process into single class

Now I think I can see the limitations of the current design. It doesn't really allow to do any cross-node optimizations or processing; if the IR node knows everything for the operation (say, lowering or an optimization), then all is good. But if it isn't, then it's really unclear what to do.
So, okay, we may try start extracting some parts of the code into a set of classes designed for batch node processing. And even (sigh) start adding some visitors.


Tumblr rate limit has been exceeded

Ripme version: 1.7.91
Java version: 1.8.0_201
Operating system: winver

Exact URL you were trying to rip when the problem occurred: artsydudejude.tumblr.com
Please include any additional information about how to reproduce the problem:

Expected Behavior
Detail the expected behavior here.
Actual Behavior
Downloading http://api.tumblr.com/v2/blog/artsydudejude.tumblr.com/posts/photo?api_key=qpdkY6nMknksfvYAhf2xIHp0iNRLkMlcWShxqzXyFJRxIsZ1Zz&offset=0
Tumblr rate limit has been exceeded
Detail the actual (incorrect) behavior here. You can post log snippets or attach log files to your issue report.

I am having the same issue. I attempted to generate a new api key too and that did not help. I clean run of RipMe on another computer that has not run RipMe has the issue too. I think this is a Tumblr issue...


Extend .nycrc exclude option with command line instead of overwriting it

Expected Behavior
.nycrc
{
  exclude: [ "foo1" ]
}

nyc --exclude foo2 npm test
Expect both foo1 and foo2 to be ignored
Observed Behavior
Only foo2 is ignored and the setting to ignore foo1 is silently removed

I have several dirs that I test separately with the --cwd option and I need to provide a different set of excludes to each different directly. This results in very redundant set of npm scripts. Ideally list of excludes should just extend each other.

@Rush have been a bit heads down working on some exciting stuff for a future release of Istanbul; Would love your help with a patch, if you're interested.

I'm running into this with tap v11 as well.  I'd like to always exclude tap-snapshots/*.js by default, but adding that to the nyc bin command line makes it re-cover test/** and friends.
Maybe there could be an additional cli or .nycrc option to add an exclude without overwriting the existing setting?

@isaacs @Rush there are definitely also situations where you want to replace the exclude rule, rather than append to the existing exclude rule. I like the idea of adding an additional command line option; what if we had something like --append-exclude; which appended rather than replacing?
@Rush any interest in implementing this?

I want to look at this but probably not in the next 2 weeks. If somebody could pick it up earlier it would be awesome
Another syntax could be --exclude +foo --exclude -bar

This is a yargs feature, right?


WebGPU: MultiRenderTarget extended support

It's the WebGPU update corresponding to #13435 for WebGL2.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13751/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13751/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13751/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13751/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13751/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13751/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13751/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13751/merge#BCU1XR#0


Added conduct file locally

Coverage remained the same at 95.876% when pulling 50914a901c3f1d878cbebb05b40f25fa01ea9a22 on feature/conduct into 761ad377e47d7b597d07235db586898b0c6aa726 on master.


Modify: PrismLauncher.PrismLauncher version 6.3

Microsoft Reviewers: Open in CodeFlow

[Policy] Reset Labels


Get all response from website with lazy loading

It executes concurrently while the page is loading page.on('response', async response => {});
I use this to scroll the page, but it does it after the page has finished loading
async function autoScroll(page){
    await page.evaluate(async () => {
        await new Promise((resolve, reject) => {
            var totalHeight = 0;
            var distance = 500;
            var timer = setInterval(() => {
                var scrollHeight = document.body.scrollHeight;
                window.scrollBy(0, distance);
                totalHeight += distance;

                if(totalHeight >= scrollHeight){
                    clearInterval(timer);
                    resolve();
                }
            }, 100);
        });
    });
}

Lazy loading demo: https://davidwalsh.name/demo/lazyload-2.0.php
Demo
Does anyone have a way? Give me an answer. Thanks!

@phanminhtai Thanks for taking the time to report this issue!
It looks like this is not actually a problem with Puppeteer.
Could ask this question on Stack overflow instead so that others can benefit from a reply too?


Add widdershins image

Add widdershins image. This image can convert openapi file to markdown.

Change directory name to chatwork/circleci-widdershins

If you are creating an image to run on CircleCI, it is better to include circleci in the directory name.

https://hub.docker.com/repository/docker/chatwork/circleci-widdershins/
Create Repository & Add permissions.


ImportError: cannot import name 'Tokenizer' from 'bm25s.tokenization' (/usr/local/lib/python3.10/dist-packages/bm25s/tokenization.py)

ImportError: cannot import name 'Tokenizer' from 'bm25s.tokenization' (/usr/local/lib/python3.10/dist-packages/bm25s/tokenization.py)

What version of bm25s are you using?

What version of bm25s are you using?
Right now, Tokenizer class is only available in 0.2.0rc8

OK. the version is Version: 0.1.10

I see. It will be included in 0.2.0, which will be release shortly. Feel free to try it in the latest prereleaae


Add World Health Organisation & France AFD & ZOA to the GB Dashboard

Adds these two organisations to the GB dashboard.

This PR was closed automatically as an unintended consequence of the set-up of the dev branch without phase 1 changes. It has now been reopened.


Allow overriding mediaHandlerFactory in UA#invite

Hi SIP.js team,
Would you please consider allowing mediaHandlerFactory to be overridden in UA#invite?
Thanks,
Mark

I think this is likely harmless.  Some part of me thinks that the media handler factory should really be tied to the UA, and that if different media wants to be handled, a different agent should handle it.  But... I don't feel strongly about it, and the UA doesn't currently depend on it at all.
Concept ACK.


Upgrade to Python 3.11

Problem
We should upgrade the Python version used in the project to Python 3.11.
Description
Python 3.11 has some nice new features and improvements so upgrading makes sense. It's also 10-60% faster (although our mileage may vary).
Changes will be needed in the following areas:

GitHub actions: actions/setup-python
Dockerfile
Pipfile
Pipfile.lock

Alternatives

We could continue to remain on Python 3.10 till it reaches EOL, although that's not a real alternative.

Marking this as blocked - #1852 took care of all services but the catalog, which is blocked by Airflow. Once Airflow supports Python 3.11, we can tackle this and close it.

Airflow supports Python 3.11 since v2.6.2, released 2023-06-17. Since we're on a newer version already, we can consider this unblocked!
https://github.com/WordPress/openverse/blob/5678bdcd64357bce1f5f390239d7a5dda97c34d3/catalog/requirements_prod.txt#L5

This has already been completed! (We're now on 3.12 😄)
https://github.com/WordPress/openverse/blob/50fe56e1a94659d91c1959f9d5d156cc7851d4e2/catalog/requirements-prod.txt#L1


Strings don't support bytes()

According to the spec, strings are supposed to support the bytes() method, but trying it in the repl fails:
$ cargo run -- -r
    Finished dev [unoptimized + debuginfo] target(s) in 0.07s
     Running `target/debug/starlark -r`
Welcome to Starlark REPL, press Ctrl+D to exit.
>>> "foo".bytes()
error[CV00]: Cannot .bytes on type string
 --> <1>:1:1
  |
1 | "foo".bytes()
  | ^^^^^^^^^^^ .bytes not supported for type string

>>>

The spec was wirtten according to the go version but the source of truth is
the Java version which does not have the bytes method. That is why it is
not in this version. The spec should probably indicate extension and
mandatory of the stdlib
Anyway that shouldn't be too hard to add and I'll be happy to do so if you
have a use case for it.
On Wed, Oct 17, 2018, 9:48 AM Jeremy Fitzhardinge<EMAIL_ADDRESS>wrote:

According to the spec
https://github.com/google/skylark/blob/a0e5de7e63b47e716cca7226662a4c95d47bf873/doc/spec.md#string·bytes,
strings are supposed to support the bytes() method, but trying it in the
repl fails:
$ cargo run -- -r
Finished dev [unoptimized + debuginfo] target(s) in 0.07s
Running target/debug/starlark -r
Welcome to Starlark REPL, press Ctrl+D to exit.

"foo".bytes()
error[CV00]: Cannot .bytes on type string
--> <1>:1:1
|
1 | "foo".bytes()
| ^^^^^^^^^^^ .bytes not supported for type string

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/starlark-rust/issues/17, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADjHf6MNFlnBK7-9UP9f2v0Zq3ArQ2Lhks5uloyAgaJpZM4Xi-XK
.

Please follow this spec: https://github.com/bazelbuild/starlark/blob/master/spec.md (it has a few differences with the other document; we plan to clarify this).

I'd say the spec should list only the mandatory functionalities. If an implementation wants to experiment new features or extensions behind a flag, it's possible - but you shouldn't expect it to work with another Starlark interpreter.

"Optional features" are not part of the language. I cannot prevent you or anyone else from adding non-standard features (you might do it for experimenting and as a proof of concept before proposing a spec change).
"Optional features" create new dialects and I'd like to avoid that. If we add a new feature to the language, it will not be optional; any compliant implementation will need to be updated.

What about feature that depends on the application? That is you don't want
them in Bazel but in some other software. E.g. in pants they want to have
generators (although I don't know why).
On Wed, Oct 17, 2018, 11:37 AM Laurent Le Brun<EMAIL_ADDRESS>wrote:

"Optional features" are not part of the language. I cannot prevent you or
anyone else from adding non-standard features (you might do it for
experimenting and as a proof of concept before proposing a spec change).
"Optional features" create new dialects and I'd like to avoid that. If we
add a new feature to the language, it will not be optional; any compliant
implementation will need to be updated.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/starlark-rust/issues/17#issuecomment-430478368,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADjHf-vDvq2wus3DKIb_3_UwiIL7cOn1ks5ulqYVgaJpZM4Xi-XK
.

I raised this because it was one of the first things I tried, and I assumed the spec was definitive.

@jsgf: sounds good, I'll close this issue as apparently it is not in the standard. I'll probably introduce a flag or a feature to add extension that are not in the spec where bytes could be.

In case you to discuss the language, clarify the semantics, or suggest new features, please do it on https://github.com/bazelbuild/starlark


Add small updates

Since we created a new discord channel #rl-i-made-this. I updated the discord.mdx + some small updates (documentation update).

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


licensing information

Hello,
can you please update this repo with the license under which you are releasing this code ?
thanks

Hello,
thanks, I see the license now.


Scala 2.13 support

Hi,
Do you plan to crossbuild gremlin-scala for scala 2.13 ?

yeah, just didn't get to it yet, life is too busy.
just tried simply upgrading a few deps - compile works, Test/compile needs some love.
:crossed_fingers: we don't need to touch the macros

Scalamock has been released for 2.13. With this version, all gremlin-scala tests pass.
I don't know why but I had to add override def getSteps: util.List[Step[_, _]] = super.getSteps in TraversalStrategySpec > RemoteGraphFixture > new AbstractRemoteTraversal to make it work.

Thanks for letting me know! And very strange indeed with the getSteps 'override'.
will merge on green: https://github.com/mpollmeier/gremlin-scala/pull/286

<IP_ADDRESS> is released for scala 2.11, 2.12 and 2.13 :tada:


Development

closes #255
closes #258

closes #249
closes #259

closes #281

closes #254

closes #251

closes #257
closes #248

@PaulaScharf regarding #296 can you add some padding so the text does not stretch so much?


3.0.3 - EntityGraphQLCompilerException: Names must only contain [_a-zA-Z0-9] but 'Byte[]' does not.

   at EntityGraphQL.Schema.BaseSchemaTypeWithFields`1..ctor(ISchemaProvider schema, String name, String description, RequiredAuthorization requiredAuthorization)
   at EntityGraphQL.Schema.SchemaType`1..ctor(ISchemaProvider schema, Type dotnetType, String name, String description, RequiredAuthorization requiredAuthorization, GqlTypeEnum gqlType, String baseType)
   at EntityGraphQL.Schema.SchemaType`1..ctor(ISchemaProvider schema, String name, String description, RequiredAuthorization requiredAuthorization, GqlTypeEnum gqlType, String baseType)
   at EntityGraphQL.Schema.SchemaProvider`1.AddType[TBaseType](String name, String description)

Guessing this is a recent change to scalar types?

Yes. As that it technically an invalid graphql name. So depending on how the tools strictly follow the spec you will get errors

From: bzbetty @.>
Sent: Friday, August 5, 2022 12:53:22 PM
To: EntityGraphQL/EntityGraphQL @.>
Cc: Subscribed @.***>
Subject: [EntityGraphQL/EntityGraphQL] 3.0.3 - EntityGraphQLCompilerException: Names must only contain [_a-zA-Z0-9] but 'Byte[]' does not. (Issue #194)
at EntityGraphQL.Schema.BaseSchemaTypeWithFields1..ctor(ISchemaProvider schema, String name, String description, RequiredAuthorization requiredAuthorization)    at EntityGraphQL.Schema.SchemaType1..ctor(ISchemaProvider schema, Type dotnetType, String name, String description, RequiredAuthorization requiredAuthorization, GqlTypeEnum gqlType, String baseType)
at EntityGraphQL.Schema.SchemaType1..ctor(ISchemaProvider schema, String name, String description, RequiredAuthorization requiredAuthorization, GqlTypeEnum gqlType, String baseType)    at EntityGraphQL.Schema.SchemaProvider1.AddType[TBaseType](String name, String description)
Guessing this is a recent change to scalar types?
—
Reply to this email directly, view it on GitHubhttps://github.com/EntityGraphQL/EntityGraphQL/issues/194, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AABGHUP3ZZL5ZP6XZLZRJJLVXR62FANCNFSM55USFU4A.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Currently working around the issue by removing byte[] scalar since i don't use it.
schemaProvider.RemoveType<byte[]>();

Correction, fixing it by adding a type since it's currently a custom mapping
schemaProvider.AddType<byte[]>("byteArray", "");

Seems HasType is not checking mappings. So Schema Builder tries to add it!


Switch all crates except tools to use Rust 2021

Edition 2021 enables some optimizations such as disjoint closure captures, along with other niceties such as an expanded prelude. This bring libtock-rs in sync with the kernel, which also uses edition 2021.
So far I just tested that make test and make examples build without warnings, and am relying on CI to catch other issues

Note that this doesn't switch the workspace to 2021, just the individual crates. E.g. the v1 feature resolver will still be in use.

My understanding is that specifying an edition for a workspace is ignored (https://github.com/rust-lang/cargo/issues/5784), and that if we want to switch to using the v2 feature resolver we should put resolver="2" in the workspace. I don't have any feelings about what resolver we should be using, however, so I am inclined to leave it as is for now.

bors r+