cannot import visualize_saliency

I'm getting this error when trying to import visualize_saliencey

I checked the Keras docs and it looks like K.image_data_format() is still valid attribute on the Keras backend. I'm wondering if this might be a Keras 1 to Keras 2 transition issue?

Yes. This lib is now incompatible with keras < 2.0.
I transitioned everything to latest.


fix: improved markdown rendering

Markdown rendering is now heavily improved. All side-effect free HTML will now be rendered. The styling also matches GitHub's markdown rendering now.
Fixes #747

Is it possible to get these changes on the home page as well? Inline code still looks not as clean for example. Seems like we have to used marked() there (for the main body of the page)

Inline code still looks not as clean for example

I'll fix that


Unable to use open with on a .jar file

Description
when I attempted to load a .jar file it would not give me an option list for the program I wanted to use instead it just opened up the folder.
Steps To Reproduce

right click .jar file
select open with
does not give you a application menu

Requirements
fix rather that loading up the folder it loads up in the appropriate program selection menu.
Files Version
<IP_ADDRESS>
Windows Version
Edition	Windows 11 Pro Version	23H2 Installed on	25/06/2024 OS build	22631.3880 Experience	Windows Feature Experience Pack 1000.22700.1020.0
Log File
debug.log

Thanks for the report, Do you have Java installed? What happens if you try in Windows File Explorer.

Hi,
Yes, Java is installed. when I do it in file explorer it gives me the
option to choose Java and it runs the application.
[image: image.png]
[image: 5cd69226-d9c5-4c77-ab35-6c3c4a2b0c92.png]
when I click the open with through files v3 it just loads up the java
folder contents.
[image: image.png][image: image.png]
Hope this helps.
Luis
On Tue, Jul 16, 2024 at 1:18 AM Josh65-2201 @.***>
wrote:

Thanks for the report, Do you have Java installed? What happens if you try
in Windows File Explorer.
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/15803#issuecomment-2229708796,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYO4Q3ZRS7MFKJBTBKUK6ETZMRREVAVCNFSM6AAAAABK5OEDD6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRZG4YDQNZZGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Email attachments don't work. You will have to click the View on GitHub link at the bottom of the email > Copy and paste or click the add attachment button to select the file > Wait until you don't see uploading in the text > Click the green comment button.

So this is what happens in windows explorer

and this is what happens in files:

Yep preview version resolves the issue and we get a pop-up box.

Great, If it happens again you can comment here.


Test case for task history cache corruption

When Gradle is aborted after removing the obsolete snapshots but before persisting the new execution snapshot then the old task history cannot be loaded any more and we may experience problems as in https://github.com/gradle/gradle/issues/2827. We already fixed the underlying problem in #2870. This PR add an integration test exposing the problem.

@big-guy I addressed your comments, could you have another look?


Expect logging._srcfile to be None

The breadcrumbs module uses the logging._srcfile attribute and crashes if it is None. The logging module sort of allows this, though:

# _srcfile is only used in conjunction with sys._getframe().
# To provide compatibility with older versions of Python, set _srcfile
# to None if _getframe() is not available; this value will prevent
# findCaller() from being called.

We use a third party library that actively sets logging._srcfile to None. I'm aware that asking raven to consider this is rather weird, but I thought I'd at least ask :smile:
I can provide a simple patch that makes _wrap_logging_method bail in that case, if there is interest.

IMO our SDK should always be defensive. I'm not sure if this is causing a hard error but:

it should only log errors and never fail
we should support odd environments to the best of our ability (do a patch here is fine)

Fixed in #1053


unable to bundle update with arel-extensions 1.6.0

Using rails 4.2.10
I note that the requirements for including arel-extensions are: Arel 6 (Rails 4) or Arel 7+ (Rails 5).
prior to including arel-extensions in the Gemfile, arel was shown in the Gemfile.lock as:

42       activerecord (>= 3.2, < 6.0)
43       rake (>= 10.4, < 13.0)
44     arel (6.0.4)
45     attr_encrypted (3.1.0)
46       encryptor (~> 3.0.0)
47     autoprefixer-rails (9.3.1)

We have arel 6.0.4.
So Gemfile amended to include:
ruby '2.5.1'
gem 'rails', '= 4.2.10'
 .
 .
 .
gem 'arel-extensions', require: 'arel/extensions'
 .
etc

in the Gemfile
and ran bundle update, which resulted in many failures, including:
Bundler could not find compatible versions for gem "arel":
  In Gemfile:
    rails (= 4.2.10) was resolved to 4.2.10, which depends on
      activerecord (= 4.2.10) was resolved to 4.2.10, which depends on
        arel (~> 6.0)

    arel-extensions was resolved to 1.5.2, which depends on
      arel (>= 7.0.0)

which seems to suggest that arel 6 might not be supported.
Could you please confirm, or, suggest a way to install against arel 6.
Thank you.

As it is rubygems which manages dependencies I think you are trying to install a different gem (https://github.com/malomalo/arel-extensions which requires arel 7).
The gem we develop can be found here https://rubygems.org/gems/arel_extensions and can be install againt arel 6.
Let us know if you encounter any issue or if you want to provide any feedback.
Regards.


Parse URL when creating TelemetryEndpoints

Closes #2803

Previously, when creating a telemetry endpoint we would not verify the validity of the provided URL. If we the URL was invalid, we would just throw a warning!, saying that the URL was invalid.
With this PR, we move this parsing logic to the creation of the TelemetryEndpoints struct.

@reviewers
Should I be adding a couple of unit tests to this? I didn't add them because the code is pretty straightforward and I felt like those tests wouldn't bring much but if you wish, I can add some.

Also this is technically a breaking change, so I guess we would wait for 3.0 to land it?
(cc @gnunicorn)

Should we merge this one as well as #5057 into one bigger PR? Might be clearer?

Tests are failing on CI but they succeeded locally on my machine. Last commits that got merged to master were failing CI too.. Not sure what the protocol is in this situation... :)

polkadot PR https://github.com/paritytech/polkadot/pull/941


First Letter Altered With PowerAccent Is Not Replaced When PowerAcccent Is Used In Google Search Bar

When typing the letter "a" in the Google search bar, the letter will not be replaced. Strangely enough, it only replaces the letters after. E.g., "a" may have "æ" typed in after, making it "aæ" in the search bar. Typing "a" once more using PowerAccent will replace the third "a" in the search bar. Now, out of the possible combinations, it could be "aææ"
To Recreate:

Type "a" in Google search bar and use PowerAccent.

See Issue

Type "a" again into the Google search bar and use PowerAccent. (This one was replaced)

I apologize, as I should have told you that this issue may occur when auto-search-prediction is in play. After some testing, it is fine when no auto-prediction is happening, but down the road most users will have things that will be auto-predicted.

Good to know. I already seen this issue before, an unfortunaly for now I don't have any idea how to resolve this for now.
But let keep in touch!

I'm reporting an observation of this issue in a different yet related place, just in case it sheds some light.
In particular, the exact same behaviour here reported is observed when I try to type an accented character (using PowerToys 0.63.0 QuickAccent) in the search bar of MS Edge (105.0.1343.53 64 bits). The offending behaviour does not happen if autosuggest is disabled.
Note that:

In Google Chrome (which I guess the original informant might be using)

A page with a Google Search text input field is shown as the "startup" or "new tab" page  (do not confuse with "Home" page). That page can be customized, but you can't remove or replace the Google Search field. This page (in its default configuration) looks quite similar to Google's website homepage, and both bring the user to a page of Google search results, but it should be noted that they're distinct from each other.
Google is usually configured as the default search engine, so that typing anything in the address bar (aka the "Omnibox") looks virtually the same and provides the same results as using the search link from the startup page.

In MS Edge, under the specific configuration I have:

The startup page is hardcoded to Microsoft Bing.
Google is configured as my default search engine to be accessed from the address bar.

Thus I can tell that I have no issue if I type in the Google Search website text box (even if autosuggest is active), nor do I have it if I type in the Bing text box at the MS Edge startup page, but I encounter the behavour reported if I type in the MS Edge address bar with autosuggest active in the browser.


PDF version of LearnOSM

User feedback from<EMAIL_ADDRESS>On http://wiki.openstreetmap.org/wiki/Beginners_Guide_1.6
There is a mention of a PDF file:

Indonesia has inspired an amazing OSM Beginner's Guide (browse or download 37-page 42.3MB PDF @ LearnOSM.org, "Beginning OpenStreetMap: A Field Guide for Everyone")

But there is no link to get any PDF file.
We should put the same file again or remove the mention in the wiki.
Also would it be worthwhile to produce and propose PDF sections from learnOSM?

But we need to find out how they were created previously if there was an automated tool.
and we should find out if we can automate their generation from markdown as well as I think gdocs versions will slowly go away as they get converted to markdown.
As nick mentioned, previously they were generated from google docs, not markdown.

Did you see this #107 :wink:
It "easy" to produce automatically pdf and/or epub version with layouts and gems :smile:

Can we agree that the mention without any link or file on http://wiki.openstreetmap.org/wiki/Beginners_Guide_1.6 can be removed?

Done.
I am closing the issue.
We should sort the remaining issues on the subject (offline read, exports, PDF, ePub, ...) and choose the best option to continue discussion.
Possibly #107. See also #56. #1 is interesting for reference and we should keep some elements in mind but we will not re-open it! ;)


Fresh project fails to run

Issue tracker is ONLY used for reporting bugs. New features should be discussed on our discord, in the #cli channel.

Expected Behavior
The app builds and the iOS simulator opens.

Current Behavior
The app errors out when trying to run web or ios.
Error: xcrun exited with non-zero code: 2 An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=2): Unable to boot device because we cannot determine the runtime bundle. No such file or directory

Possible Solution

Steps to Reproduce

Context (Environment)
node -v: 20.4.0
pnpm -v: 8.15.3
Project configuration:
{
  projectName: 'merge-ai',
  packages: [
    {
      name: 'expo-router',
      type: 'navigation',
      options: { type: 'stack' }
    },
    { name: 'nativewind', type: 'styling' }
  ],
  flags: {
    noGit: false,
    noInstall: false,
    overwrite: false,
    importAlias: true,
    packageManager: 'pnpm'
  }
}

Detailed Description

Possible Implementation

Resolved by updating Xcode.


TypeError: load() got an unexpected keyword argument 'transform_type'

@po0ya @kabkabm . Thank you very much for the work .
I am trying to create deter physical adversarial attacks using defensegan and trained the GAN on celebA dataset. However, when I tried to reproduce the blackbox of whitebox attack  I am stuck at this error. "TypeError: load() got an unexpected keyword argument 'transform_type'"
Any idea how to fix this ?  I am running the code on a google cloud vm with Cuda 7.05 . Trained the gan up to 135000 iterations .
**2018-11-04 04:24:57.427800: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions tha
t this TensorFlow binary was not compiled to use: AVX2 FMA
2018-11-04 04:24:59.468111: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:964] successful NUMA node read f
rom SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2018-11-04 04:24:59.468505: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1411] Found device 0 with properties
:
name: Tesla K80 major: 3 minor: 7 memoryClockRate(GHz): 0.8235
pciBusID: 0000:00:04.0
totalMemory: 11.17GiB freeMemory: 11.10GiB
2018-11-04 04:24:59.468537: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1490] Adding visible gpu devices: 0
2018-11-04 04:24:59.777777: I tensorflow/core/common_runtime/gpu/gpu_device.cc:971] Device interconnect StreamExecu
tor with strength 1 edge matrix:
2018-11-04 04:24:59.777830: I tensorflow/core/common_runtime/gpu/gpu_device.cc:977]      0
2018-11-04 04:24:59.777839: I tensorflow/core/common_runtime/gpu/gpu_device.cc:990] 0:   N
2018-11-04 04:24:59.778142: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1103] Created TensorFlow device (/jo
b:localhost/replica:0/task:0/device:GPU:0 with 10758 MB memory) -> physical GPU (device: 0, name: Tesla K80, pci bu
s id: 0000:00:04.0, compute capability: 3.7)
[*] Checkpoint is read successfully from output/gans/celeba
2018-11-04 04:24:59.805142: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1490] Adding visible gpu devices: 0
2018-11-04 04:24:59.805183: I tensorflow/core/common_runtime/gpu/gpu_device.cc:971] Device interconnect StreamExecu
tor with strength 1 edge matrix:
2018-11-04 04:24:59.805200: I tensorflow/core/common_runtime/gpu/gpu_device.cc:977]      0
2018-11-04 04:24:59.805210: I tensorflow/core/common_runtime/gpu/gpu_device.cc:990] 0:   N
2018-11-04 04:24:59.805428: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1103] Created TensorFlow device (/jo
b:localhost/replica:0/task:0/device:GPU:0 with 10758 MB memory) -> physical GPU (device: 0, name: Tesla K80, pci bu
s id: 0000:00:04.0, compute capability: 3.7)
Traceback (most recent call last):
File "blackbox.py", line 762, in
tf.app.run(main=main_cfg)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 125, in run
_sys.exit(main(argv))
File "blackbox.py", line 761, in
main_cfg = lambda x: main(cfg, x)
File "blackbox.py", line 685, in main
defense_type=FLAGS.defense_type)
File "blackbox.py", line 419, in blackbox
get_cached_gan_data(gan, test_on_dev, orig_data_flag=True)
File "blackbox.py", line 357, in get_cached_gan_data
orig_data=orig_data_flag,
File "blackbox.py", line 245, in get_celeba
ds_test.load(split=dev_name, transform_type=1)
TypeError: load() got an unexpected keyword argument 'transform_type' **

Hi @bibin-sebastian, thanks for pointing this out!
Unfortunately, we don't have the right environment/data setup to test the fix, can you please try get_celeba_fix branch (git pull origin get_celeba_fix and git checkout get_celeba_fix) and see if this is fixed. This is due to an obsolete argument transform_type for the load function. Let us know if more fixes are needed to get celeba running. Will merge the branch into master after making sure that it works fully.

@po0ya .Thanks for the reply . tried that and hit two more issue. The first issue was with "dev_name  = 'dev'" variable which did not have a matching description  in CelebA.py. So changed dev_name to "val". Then the next issue where I am stuck badly  is --- 'LazyDataset' object has no attribute 'shape'

My need is to run a whitebox attack on model A . the file that was run --> whitebox.py.
Error description below ------------
name: Tesla K80 major: 3 minor: 7 memoryClockRate(GHz): 0.8235pciBusID: 0000:00:04.0totalMemory: 11.17GiB freeMemory: 11.10GiB2018-11-04 19:56:36.788679: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1490] Adding visible gpu devices: 02018-11-04 19:56:37.093072: I tensorflow/core/common_runtime/gpu/gpu_device.cc:971] Device interconnect StreamExecutor with strength 1 edge matrix:2018-11-04 19:56:37.093135: I tensorflow/core/common_runtime/gpu/gpu_device.cc:977] 0 2018-11-04 19:56:37.093148: I tensorflow/core/common_runtime/gpu/gpu_device.cc:990] 0: N 2018-11-04 19:56:37.093444: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1103] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 10758 MB memory) -> physical GPU (device: 0, name: Tesla K80, pci bus id: 0000:00:04.0, compute capability: 3.7) [*] Checkpoint is read successfully from output/gans/celebaTraceback (most recent call last): File "whitebox.py", line 395, in  tf.app.run(main=main_cfg) File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 125, in run _sys.exit(main(argv)) File "whitebox.py", line 394, in  main_cfg = lambda x: main(cfg, x) File "whitebox.py", line 292, in main num_train=FLAGS.num_train, File "whitebox.py", line 106, in whitebox x_shape = [None] + list(train_images.shape[1:])AttributeError: 'LazyDataset' object has no attribute 'shape'

Hi , Thanks for the support. We have another error after changing the two files.

please pull again

Hi . Sorry for the trouble . Here is another one

please pull again

Hi @po0ya  we had to switch VM due to CUDA 7.21 so it took a bit of time to run the code after switching the VM . It felt like the code ran for some time since I could see that GPU (1300MB)  & CPU utilization was up .However, we hit another road block.

np, please pull

Hi. another similar error

please pull,
[now I remembered that this file is an older version of the file for some reason, from before the deadline! if it's not fixed in this conversation, I'll go find the latest version of this file.]

Hi. Thanks very much. This looks promising. I will update you soon once the whole iteration is complete.

Hi. @po0ya . Please find the update.

It would be highly appreciated if you can look for the correct file. Thank you.

closing this since resolved in offline discussions


Landing page improvements

@roytomeij  - not 100% sure on the tone of voice here, I think we can maybe refine it more - but I wanted to make this page more welcoming and clear
@wesoudshoorn - I added a CTA button for the guides... did I break branding rules?

@roytomeij - capitalization, what is our guide on this? It's not covered in our handbook, and I notice we capitalize the first word "Your application metrics"
I'm an advocate for the following rules: https://www.webucator.com/article/how-to-capitalize-headings-and-titles/
If this is against the tone / style guide, I'd revise it to keep current standards but always capitalize feature names (everywhere) - we don't currently do that, and it can make it hard at a glance to see that a specific term is contained within a title/link.

We actually have strong editorial guidelines for the blog, and I think since Connor owns the docs, we can use the same style there.
https://www.ox.ac.uk/sites/files/oxford/media_wysiwyg/University of Oxford Style Guide.pdf
This is basically the same thing Connor has been suggesting, this is an adaptation of Chicago Manual of Style with easy-to-understand examples. I'm all up for it. :)


Add FINOS Active Badge to App Integrations Commons project README.md

Description
This pull request removes the projects Incubating badge and adds the FINOS Active project badge to the project's README.md to reflect the Active status of the project within the FINOS foundation.
FINOS Active Badge

Markdown Added
[![FINOS - Active](https://cdn.jsdelivr.net/gh/finos/contrib-toolbox@master/images/badge-active.svg)](https://finosfoundation.atlassian.net/wiki/display/FINOS/Active)

Can one of the admins verify this patch?

Hey there,
Following raising this PR on 4th October 2020, the FINOS team will merge this PR on Friday 16th October 2020, so the project status is correctly represented in the README.md
Please feel free to merge the PR direct or allow the FINOS team to merge on your behalf.
James.


ndstore/statuspage_config

@Aeusman I have now created statuspage_config. Let's move everything here and populate the README in that folder for setup instructions(TODO in there for you)

TODO for us is to also add a cron job on this.

Cron job added.


pipeline section (WIP)

Merge main to this branch and add this section in delivery-engineer/pipelines/tekton


Fix grammar in proof-of-transfer.md

"Additionally, By" => "Additionally, by"

Thanks @lukechilds  for your contribution!
We will merge your change as soon we merge this other PR that isn't allowing some checks to pass.
@jhammond2012, lets merge that one first and then re-run the checks here.

@criadoperez and @lukechilds any ideas on this one?


Show search bar in searchable views by default

Searching is currently initiated by starting to type while a searchable view is focused. In my opinion, the added clarity offered by showing the search bar by default outweighs the “lost” screen real estate.

This is added a while ago


Javascript arguments

Splitjoins javascript function arguments.
Implementation is simply done by extracting and reusing functions already present inside js.vim.  It might be even more generalizable by moving these helper functions to the main sj helper 'module', but this decision I would leave to you, @AndrewRadev.
Also fixes minor issues with semicolon handling in javascript. Splitjoining functions was producing js that wouldn't make it through a linter like jshint.
Note that this currently sits on top of #58 and should not be merged. Depending on the fate of #58 I would rebase or cherry-pick this accordingly before taking further action.
Closes #53

Looks good to me. I did something similar for python lists/tuples, and I left the helpers as script-local functions.
I don't indent to extract them, since there are some javascript-specific things, particularly indentation. I'd say this is good enough as it is.
Thanks for your work :).

I've had to make some changes to javascript if-clause handling due to issue #67.
Long story, short, I now have a branch called if-clauses-with-curly braces in which I've overwritten the javascript if-clause handling with the PHP one (since I wanted to unify them and I had to pick one and this one was implemented by me :P). This may mean that I've made mistakes in the implementation and I'd appreciate it if you could take a look (either at the implementation, or just try it out and see if it seems to work alright).
Also note that there's a variable now, g:splitjoin_javascript_if_clause_curly_braces, which controls whether curly braces are added or not upon splitting/joining. Its value is not a boolean flag as usual, but I've documented it, so I hope it's understandable how it works. Otherwise, do let me know :).


ci: add test

Is lua busted redundant with our vendored deps?

I get the following error if I don't install lua-busted:
/usr/bin/lua5.1: deps/share/lua/5.1/busted/runner.lua:27: module 'busted' not found:No LuaRocks module found for busted

I have no experience with luarocks so I don't know if this means it's not redundant or if some environment variable/path needs to be adjusted.


Update badge colors to pass contrast minimums

Validated against the XD file and checked with storybook accessibility plugin. These should all be static colors that don't change with the theme.

The failing chromatic seem unrelated to this PR, they can be fixed elsewhere, so let's merge this despite the chromatic tests.


Inconsistent Graph Tracing Line Width

Line widths of auxiliary lines in the main graph are not unified.

Fixed.


test: add bib anaconda-iso test

This PR added bib anaconda-iso image building test.

/test-rhel95-bib-image

/test-cs9-dev-bib-image

/test-rhel95-replace

okey, no regression. Merged.


Support Shortcut to start vehicle

Is your feature request related to a problem? Please describe:
When using the "Toggle Accessory State" shortcut in the shortcuts app, nothing happens.
Describe the solution you'd like:
When using the "Toggle Accessory State" option, a remote start (or stop) command is sent.

Here is the log entry for when this shortcut is run:
[17/06/2024, 13:16:56] [KiaConnect] getBackLeftContactSensorState: 0
[17/06/2024, 13:16:56] [KiaConnect] getOn:  false
[17/06/2024, 13:16:56] [KiaConnect] getBackRightContactSensorState: 0
[17/06/2024, 13:16:56] [KiaConnect] getHoodContactSensorState: 0
[17/06/2024, 13:16:56] [KiaConnect] getCurrentLockState: 1
[17/06/2024, 13:16:56] [KiaConnect] getTargetLockState:  1
[17/06/2024, 13:16:56] [KiaConnect] getTrunkContactSensorState: 0
[17/06/2024, 13:16:56] [KiaConnect] getCurrentTemperature: NaN
[17/06/2024, 13:16:56] [KiaConnect] getStatusBatteryLow: false
[17/06/2024, 13:16:56] [KiaConnect] getOccupancyDetected: false
[17/06/2024, 13:16:56] [KiaConnect] getFrontLeftContactSensorState: 0
[17/06/2024, 13:16:56] [KiaConnect] getFrontRightContactSensorState: 0
[17/06/2024, 13:16:56] [homebridge-kia-connect] This plugin generated a warning from the characteristic 'Current Temperature': characteristic value expected valid finite number and received "NaN" (number). See https://homebridge.io/w/JtMGR for more info.

UPDATE: To use the shortcut, go to accessory settings in Home and select "Show as different tiles". You can then create separate shortcuts for start/stop and lock/unlock.


No way to distinguish between target related metrics and exporter metrics

According to documentation some metrics like idrac_exporter_build_info and idrac_exporter_scrape_errors_total relates not to scraped target but to exporter itself. With provided prometheus relabeling config instance label will be added to such metrics so instead of one idrac_exporter_scrape_errors_total we will have this metric for every target.
It would be great to export all target related metrics with label target and exporter related metrics without such label. Also in this way any additional relabeling on prometheus side will be not required.

Hi @dmitry-sinina
The idrac_exporter_scrape_errors_total is target specific (the number of scrape errors for the specific target). The idrac_exporter_build_info is indeed the same for all targets. However, I do not see any real value in changing the exporter because of a single metric being repeated for every target. You normally have this kind of information from every target anyway, it's just in this case it comes from the same binary (but some people do run one instance of the exporter per target).


feat: display basic yield generator report

Display the yield generator report as simple table.

Needs a better representation but I suppose it is better than nothing. Hopefully someone with a sense for design can improve the visualization.
Closes #47.

Should we display it in useAdvancedWalletMode only for now?

Should we display it on useAdvancedWalletMode only for now?

Or make it toggleable on the earn page. Something like a "show report" button maybe?

Should we display it on useAdvancedWalletMode only for now?

:heavy_check_mark: I don't want to make designers unhappy. I guess we get away by displaying it only in "advance" mode.

Or make it toggleable on the earn page. Something like a "show report" button maybe?

"Display report" / "Show report"?

Or make it toggleable on the earn page. Something like a "show report" button maybe?

I'd use a button for that. A toggle (which is basically just a glorified checkbox) for showing hiding UI is not really common afaik.

I'd use a button for that. A toggle (which is basically just a glorified checkbox) for showing hiding UI is not really common afaik.

Nice! Looks good to me +1 One small suggestion to improve the error wording.

:+1:

What's the easiest way to generate data for this report on regtest?

You can start the marker service from a wallet with funds (e.g. after ./init-setup.sh): Then you will immediately see the "Connected" line.
To get actual data you'd do a coinjoin and then change the target in ./setupProxy.js to route /api/ to https://localhost:29183 , which is the second instance that coinjoined with you. I think there is no live reload so you'd need to run npm start again.
PS: I want to use our standalone docker image as base for the secondary instance in the future, then we can have the most recent version on port 8081 and do better regression testing.


Module not found: Error: Can't resolve './components/context'

ui build is giving the module not found error.
Module not found: Error: Can't resolve './components/context' in '/opt/network-orchestration-for-aws-transit-gateway/deployment/regional-s3-assets/ui/src'

Got the same error on V3.3.3 as well.

@groverlalit Any suggestions on how to fix this?

I am not able to duplicate this issue.  I built it locally and checked in the build pipeline. It finished building with warning. Please see screenshot below. Are you following the steps in README? If not, can you please share steps to duplicate this issue? Thanks for your patience.

Yes. My local environment is Windows, so I launched an AL 2023 EC2 instance and performed the steps as per the Readme file.
The build should have failed unless you are using a Macbook. As you can see in below screenshot the import statement is "components" whereas the directory is "Components"

Thanks for finding the root cause. Are you able to build it if you change the directory name?

The directory name has been fixed on the release/v3.3.3 branch. Thanks

Thanks. The fix resolved the issue.


Existing grassroots repositories

Let's use this thread to keep track of similar and relevant efforts.

Supernovae - https://www.wiserep.org/
ExoAtmospheres - https://research.iac.es/proyecto/exoatmospheres/
Transiting planets - https://www.astro.keele.ac.uk/jkt/tepcat/
Starchive - not yet live.

https://astrocats.space/ for supernovae, kilonovae, TDEs, fast stars and exoplanets
https://gucds.inaf.it/gucds/login.php for spectra and general data in the Gaia Ultracool Dwarf Sample (not yet live)
https://www.ipac.caltech.edu/projects/category/archives-and-data-science for exoplanets and extragalactic and IRSA bits

NASA exoplanet archive: https://exoplanetarchive.ipac.caltech.edu/

Hypatia: https://www.hypatiacatalog.com/

species: https://species.readthedocs.io/en/stable/index.html

Community Atlas of Tidal Streams (CATS): https://github.com/stellarstreams/cats
RR Lyrae - Cecilia Mateu - private collection of tables. 3 CSV files.  @cmateu

https://www.wiserep.org/aux/

gas opacities:
HITRAN: https://hitran.org/data-index/
exomol: https://www.exomol.com/
Dace: https://dace.unige.ch/opacityDatabase/

ExoplANETS-A: https://arxiv.org/abs/2405.06577

https://mocadb.ca/ - Montreal Open Clusters and Associations (MOCA) database.

coming soon: SUCANES database of proto-brown dwarfs.


Support input objects

We have several constraints:

Should be easy to use - this points to using a Haskell data type over e.g. a Map Text Value.
Input must be validated against a definition: This allows both a type-class checking against Object name i f and a derive Generic record.
Definition itself must be valid in the GraphQL sense. This can be derived both from an Object name i f and a derive Generic record.

Would be nice but not necessary:

Symmetry with handler definitions, i.e. using Object n i f to define input objects as well.
Make defaults work somehow, but we have this issue already, and I'm not even sure how it's meant to work when reading e.g. https://github.com/graphql/graphql-js/issues/385

I think 1 in particular points to a derive Generic style input. We could add that to FromValue.

What exactly do you mean by "support input objects"?  To the extent that we run fromValue on data received from users, we already do this.
Also, "input objects" is a technical term in GraphQL, referring to a restricted type of Object suitable for input.

The link is exactly what I mean. I'm not sure where we read input objects? Do we have a generic class already? Can't see it in the code.

https://github.com/jml/graphql-api/blob/master/src/GraphQL/Resolver.hs#L265

Discussed IRL: There will be no extra metadata other than the data that can be derived generically.

This is implemented though the API probably isn't final (see #73)

Just to be clear for googlers and to save having to click elsewhere: GraphQL specifies two types of Objects.

GraphQLObjectType
GraphQLInputObjectType

At first glance, the two types might seem redundant.  But, it turns out there is value in having a distinction between your inputs and outputs for the same object.
The current state of the graphql-api only derives generic HasAnnotatedInputType for records with 3 or less fields.


IndexError: list index out of range when running benchmark on gguf model

I am running a llamacpp server on local host with phi-2_16.gguf model (as described here: https://github.com/EleutherAI/lm-evaluation-harness/issues/1254).
When running the lm-eval test suite (version 0.4.2) with the following commands:
lm_eval --model gguf --tasks winogrande --model_args base_url=http://localhost:8080 --output_path ./output/phi-2/winogrande --verbosity DEBUG --num_fewshot 5
the execution breaks during the loglikelihood requests (for the 259th request) with the following exception:
File "<path_to_repository>\lm-evaluation-harness\lm_eval\__main__.py", line 341, in cli_evaluate
    results = evaluator.simple_evaluate(
              ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File <path_to_repository>\lm-evaluation-harness\lm_eval\utils.py", line 288, in _wrapper
    return fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^
  File "<path_to_repository>\lm-evaluation-harness\lm_eval\evaluator.py", line 251, in simple_evaluate
    results = evaluate(
              ^^^^^^^^^
  File "<path_to_repository>\lm-evaluation-harness\lm_eval\utils.py", line 288, in _wrapper
    return fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^
  File "<path_to_repository>\lm-evaluation-harness\lm_eval\evaluator.py", line 390, in evaluate
    resps = getattr(lm, reqtype)(cloned_reqs)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<path_to_repository>\lm-evaluation-harness\lm_eval\models\gguf.py", line 89, in loglikelihood
    logprob, is_greedy = get_result(logprobs, len(context))
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<path_to_repository>\lm-evaluation-harness\lm_eval\models\gguf.py", line 22, in get_result
    while offsets[idx] < context_length:
          ~~~~~~~^^^^^
IndexError: list index out of range

I can't figure out what the problem is, am I using this incorrectly?

i meet the similar issue:
i use lm-eval (0.3.2) run agieval:
lm_eval --model vllm \         --model_args pretrained=${model},trust_remote_code=True,tokenizer_mode="slow",tensor_parallel_size=8,dtype=auto,gpu_memory_utilization=0.8 \         --tasks agieval \          --batch_size auto \         --output_path ${x} \         --num_fewshot 5 \         --device cuda      done
getting:
Traceback (most recent call last):   File "/usr/local/bin/lm_eval", line 8, in <module>     sys.exit(cli_evaluate())   File "<path_to_repository>/lm_eval/__main__.py", line 342, in cli_evaluate     results = evaluator.simple_evaluate(   File "<path_to_repository>/lm_eval/utils.py", line 288, in _wrapper     return fn(*args, **kwargs)   File "<path_to_repository>/lm_eval/evaluator.py", line 234, in simple_evaluate     results = evaluate(   File "<path_to_repository>lm_eval/utils.py", line 288, in _wrapper     return fn(*args, **kwargs)   File "<path_to_repository>/lm_eval/evaluator.py", line 325, in evaluate     task.build_all_requests(   File "<path_to_repository>/lm_eval/api/task.py", line 418, in build_all_requests     fewshot_ctx = self.fewshot_context(   File "<path_to_repository>/lm_eval/utils.py", line 288, in _wrapper     return fn(*args, **kwargs)   File "<path_to_repository>/lm_eval/api/task.py", line 950, in fewshot_context     labeled_examples = description + self.sampler.get_context(doc, num_fewshot)   File "<path_to_repository>/lm_eval/api/samplers.py", line 37, in get_context     [   File "<path_to_repository>/lm_eval/api/samplers.py", line 49, in <listcomp>     str(self.doc_to_target(doc)[0]) IndexError: list index out of range
i don't know the reason and i change the --num_fewshot 0 it works. but i don't know why num_fewshot can't be 5.
and can someone teach me how to set up num_fewshot for different datasets?

I faced a similar issue.
In my case, the problem was that llama-cpp-python assumes that the prompt will always inlcude a BOS token.
This resolves my issue: https://github.com/abetlen/llama-cpp-python/pull/1471.


Don't swallow other flavors of botocore.exceptions.ClientError on failed pw change

@rhdedgar gave 👍 in stg PR #2048. That's right, I got PR # 2^11!

[test]

[test] again please


Filter: load JSON definitions and policies from file

Why:
We want users to have access to the full Smorgasbord of configuration including different encryption policies per-topic and support for multiple KMS implementations.
With this change the Kroxylicious Filter will require configuration to point to the two JSON files. This gives us the closest integration with the encryption module because JsonPolicyLoader.loadTopicPolicies works in terms of File objects and does some useful work like validating and associating policies and KMS implementations.
example Kroxylicious configuration
filters:
  - type: TopicEncryption::EncryptProduce
    config:
      kmsDefinitionsFile: "/path/to/kms-def.json"
      topicPoliciesFile: "/path/to/topic-policies.json"
  - type: TopicEncryption::DecryptFetch
    config:
      kmsDefinitionsFile: "/path/to/kms-def.json"
      topicPoliciesFile: "/path/to/topic-policies.json"

In future, if we want to be able to embed the KMS definitions and topic policies in the Filter configuration it might be nice if JsonPolicyLoader had another method like:
List<TopicPolicy> loadTopicPolicies(List<TopicPolicy> topicPolicies, Map<String, KmsDefinition> kmsDefs) that did the same work without the dependency on loading JSON from File.
Implements #3

I've now refactored the tests, splitting them into tests covering Produce Encryption and Fetch Decryption, testing each side in isolation. This combined with adding tests for per-topic encryption policies would catch the issues fixed by commit #de60abe. The tests were working because of the hardcoded wildcard policy combined with the metadata request failure caching an empty string. We found a name for the topic (empty string), and that got the wildcard policy applied to it, test passes :heavy_check_mark:

To catch a comment from the demos, @SamBarker pointed out that it would be good if we could get credentials into the key manager definition file (like the vault token) in a kubernetes friendly way. So instead of directly in our JSON file, refer to a secret file or environment variable. Captured in #8


fixFix the need for low-level table permissions when querying…

… views in certain situations (#44621)
pick: https://github.com/apache/doris/pull/44621

run buildall

run feut

run feut


Version 1.1.6 has a dependency CSRF vulnerability

Hello,
I am trying to use the NPM package of Cortex, but after installation it shows me that is uses a version of axios that has a CSRF vulnerability and npm suggests that i should make the npm audit fix --force command, but this way Cortex verion 0.0.10 will be installed, which is not what I need or wnat.
Is there any way you can update the dependencies in the package, so Ieveryone could use the latest version an not the npm suggested 0.0.10 version of Cortex?

bump

hi @andrejmoltok, thank you for the feedback looking into this, created a pr https://github.com/aj-archipelago/cortex/pull/124 next version should have the fix

1.1.7 should fix it, please let us know if you still have issues


Fix Scoreboard

🎮 feature Request
feature logic and basic description
When we start the game score board overlaps the player details page.

Point down the features
feature points
Remove scoreboard till the game does not starts and when the game starts put it back.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey I wants to contribute in this  works please assign me the work so that I can contribute in this..

@ayush-t02 Also, when playing with 6 players - the scoreboard doesn't fit in the page and creates scroll.
I want to fix the scoreboard position and alignment.


Fix editing an existing box

Doesn't work at all right now, includes

Changing name
Changing details
Changing photo
Deleting box
etc.

#32

API changes required as well
https://github.com/sensebox/OpenSenseMap-API/commit/4cff7c96ebf644c7a059b7c4fc4364ae62696fa8

Photo is still not displayed correctly (broken link). Should this work? It's checked above.

It worked at some point... weird, thanks for letting us know

Editing may also include change of sensor setup

I uploaded a revised version on http://demo.opensensemap.org, editing sensors does not work at this point but I'm working on it. Let me know what you think.

The photos are now displayed. Thank you.
I had a look at the demo. It looks good.

It's now done, will be online soon


Orphaned pages do not enforce uniqe title

@greg0ire this is the orphan tag that I told you about. With this change you can just add the :orphan: tag at the beginning of those files that are not displayed in the doctree and need no title.


Conditions with a value can no longer be increased

Pretty sure you guys are already aware of this, but decided to make an issue to be safe.
Cannot increase the value of any condition with a value, i.e. stupefied, stunned, enfeebled, etc.
PF2e version: 5.8.2
Core: 11.314

Are you trying to increase the condition value via rule element?

No, trying to increase it through the effect/conditions bar
See here
If you left click nothing happens

It seems to work if you have increased it from the hud first though.


Bump knative to 0.22 🧙

Changes
Keeping knative up-to-date.
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>/kind misc
/cc @afrittoli @sbwsg
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Docs included if any changes are user facing
[ ] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes
Update knative dependency to 0.22, and default minimum kubernetes version supported, v1.18.0

This will require upgrading our infra clusters :)

This will require upgrading our infra clusters :)

@afrittoli are we still running 1.17 ?

This will require upgrading our infra clusters :)

@afrittoli are we still running 1.17 ?

Yes, I updated them to 1.17 in December, so it's now time to a 1.18 update

Fixes #3720 🎉  :D

/lgtm

/retest

You'll need to rebase on top of main to get the integration tests passing.

git fetch upstream
git rebase upstream/main
git push --force <your remote> <branch name>

thank you @vdemeester
/lgtm


(SC-CEN-4) - There is not a field to set de description

There is not a field to set the entity description.

All basic objects - layers, entities, models and risk configurations - have now a description field that can be modified


WAVE accessability tool reports errors on empty TH

Using WebAIM's WAVE tool (extension for Google Chrome) to evaluate the accessibility of websites, floatThead is causing an issue because of empty TH. You can replicate this by going to the main product page (http://mkoryak.github.io/floatThead/) and running the WAVE tool. You will see 10 errors, one for each empty TH for each table. Now each of those THs have an aria-description so I think it may not really be an error in the code altering the THEAD, but it is flagged as an error. Maybe don't take out the text in each TH?
Google Chrome, Windows 7.

I take the text out of the TH because I can't predict how it will be formatted with the user's css. If they have some crazy rules in place that changes the size of the thead, it will break the plugin.
aria-description is the standard for accessibility. I am not going to potentially break the plugin because a tool doesn't think that using a standard is enough.

There is a workaround. I used to do this text copying thing before (it turned out to be a problem because of css rules issues). If you use release 1.4.2  https://github.com/mkoryak/floatThead/releases
you can use enableAria flag to enable this functionality. That flag has been removed in later versions and will not be re-added.


How to use react router in dart?

Help!!!  How to use router in react-dart?

@wuquanyao you would have to write a JS interop for it.

@aaronlademann-wf it might be a bit late for you, but I have successfully implemented React Router with this library which I will document here for anyone else who needs this info.

Import ReactRouter (I am using webpack) and creat JS objects inside a wrapper

// React Router
import {
  BrowserRouter as Router,
  Switch,
  Route,
  Link
} from "react-router-dom";

window.ReactRouterWarper = {
  _Router: Router,
  _Switch: Switch,
  _Route: Route,
  _Link: Link,
}

Create a react_router.dart which creates the Dart instances of the ReactRouter Components

@JS('ReactRouterWarper')
library wrapper;

import 'package:js/js.dart';
import 'package:react/react_client.dart';
import 'package:react/react_client/react_interop.dart';

@JS()
external ReactClass get _Router;
external ReactClass get _Switch;
external ReactClass get _Route;
external ReactClass get _Link;

final Router = new ReactJsComponentFactoryProxy(_Router);
final Switch = new ReactJsComponentFactoryProxy(_Switch);
final Route = new ReactJsComponentFactoryProxy(_Route);
final Link = new ReactJsComponentFactoryProxy(_Link);

Use ReactRouter in your code - first create a top level Router

main() {
  setClientConfiguration();
  render(MainRouter({}), querySelector('#mount_point'));
}

var MainRouter = registerComponent(() => new _MainRouter());
class _MainRouter extends Component2 {
  render() {
    return Router(
      {},
      Switch(
        {},
        Route(
          {'path': '/login'},
          Login({}),
        ),
        Route(
          {
            'path': '/',
            'exact': true,
          },
          Home({}),
        ),
      ),
    );
  }
}

Then to create a link, use the Link Component (in this case I am also using MaterialUI)

_appBar() {
    return AppBar(
      {'position': 'static'},
      ToolBar(
        {},
        Grid(
          {
            'container': true,
            'direction': 'row',
            'justify': 'flex-end',
          },
          Button(
            {'color': 'inherit'},
            Link({
              'to': '/login',
              'className': 'white-link-no-decoration',
            }, "login"),
          ),
        ),
      ),
    );
  }

One final consideration, is that webdev serve returns a 404 for any route it cannot find, which is no use when you refresh a page with a route as you want your client-side code to resolve the link. Using webdev_proxy serve solves this problem as all 404s are redirected back to index.html and everything works perfectly.

@barriehadfield have you published your library to pub?

Hi @aaronlademann-wf - the solution does not warrant a library as it is as simple as stated above. Your wonderful library made it really easy.
I would be more than happy to add to your example collection and issue a PR if that would be useful?

I'll leave it up to you if you want to contribute an example.
As far as webdev 404 handling is concerned... that's why our very own @evanweible-wf wrote webdev_proxy! Its works great - we use nothing but the proxy within our internal apps at Workiva.

OK great. I will wait for your new release and issue a PR thereafter to avoid any potential merge conflicts.
The examples were instrumental in getting me started, so I think having ReactRouter covered would be good.
webpev_proxy is key! Thank you @evanweible-wf

If anyone else comes across this issue, I have created a library which includes the bindings that @barriehadfield created as well as the built js code to include in the codebase.
Repo can be found here: https://github.com/matthewnitschke/react_router_dart
And its published to pub here: https://pub.dev/packages/react_router
The library includes bindings for both react-dart and over_react


Added the London Spektrix event

Added the London event, based off Fleet Street in central London at the Spektrix London Offices.

Welcome! 🎉

Thank you. 😀
On Fri, 6 Oct 2017 at 10:09, Raimo Radczewski<EMAIL_ADDRESS>wrote:

Welcome! 🎉
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coderetreat/coderetreat.github.io/pull/141#issuecomment-334701338,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ae_T2EOeTyHarzjpLeudbKhPbMK3q5c1ks5spe5GgaJpZM4PwKY-
.


include/openssl/applink.c missing in Windows builds

See #16570
Version: 3.0.1
Platform: Windows
CMake Error at C:/a/1/s/scripts/buildsystems/vcpkg.cmake:572 (_add_library):
  Target "lua-ssl" links to target "OpenSSL::applink" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?

Configuration
Then target_link_libraries([...] PRIVATE OpenSSL::applink)
Note that the configuration and OpenSSL::applink work with OpenSSL 1.1.1m.

I just built OpenSSL 3.0.1 for Windows, and I can find include/openssl/applink.c
[fspatar@nx-server-x64c:~/new/trunk]$ unzip -t OPSWopenssl_fips-<IP_ADDRESS>-x64.zip | grep "applink.c"
    testing: OPSWopenssl_fips/<IP_ADDRESS>/x64/include/openssl/applink.c   OK

I can also see the file being copied in the build logs:
[...]
Thu Feb 24 04:59:29 2022:I  _rCI: *** Installing development files
Thu Feb 24 04:59:29 2022:I  _rCI: created directory `F:/build/fspatar/win32/6.3-X64/build/OPSWopenssl/install/OPSWopenssl_fips/<IP_ADDRESS>/x64/include'
Thu Feb 24 04:59:29 2022:I  _rCI: created directory `F:/build/fspatar/win32/6.3-X64/build/OPSWopenssl/install/OPSWopenssl_fips/<IP_ADDRESS>/x64/include/openssl'
Thu Feb 24 04:59:29 2022:I  _rCI: Copying: ./ms/applink.c to F:/build/fspatar/win32/6.3-X64/build/OPSWopenssl/install/OPSWopenssl_fips/<IP_ADDRESS>/x64/include/openssl/applink.c
Thu Feb 24 04:59:30 2022:I  _rCI: Copying: ./include/openssl/aes.h to F:/build/fspatar/win32/6.3-X64/build/OPSWopenssl/install/OPSWopenssl_fips/<IP_ADDRESS>/x64/include/openssl/aes.h
[...]

I just built OpenSSL 3.0.1 for Windows, and I can find include/openssl/applink.c
[fspatar@nx-server-x64c:~/new/trunk]$ unzip -t OPSWopenssl_fips-<IP_ADDRESS>-x64.zip | grep "applink.c"
    testing: OPSWopenssl_fips/<IP_ADDRESS>/x64/include/openssl/applink.c   OK

It looks like you built it on Unix. However, I built in on my Windows machine and I can't find it.

I further inspected CI logs and I could find it. Vcpkg builds openssl on its own. Is there anything in the build system that changed since 1.1.1m and may require adjusting our build system?

However, I've found that Cmake's FindOpenSSL.cmake is making certain assumptions on Windows, it seems to be made to match a certain binary distribution with an installation layout that isn't ours

I had a look at FindOpenSSL.cmake, and indeed found that there was a bug that could result in this behavior.  It was fixed a few months ago:
https://gitlab.kitware.com/cmake/cmake/-/commit/586d483983a963ad2a73511080f591bc4aea8578
According to their current git history, this hasn't made it into a release yet, but will be part of Cmake 3.23, which is currently out as release candidate 2.
It may be that you'll have to patch your copy of FindOpenSSL.cmake, or build your own Cmake from their current release branch

@levitte Because CMake can't build the target applink is it possible to build it manually using CMake?

I had a look at FindOpenSSL.cmake, and indeed found that there was a bug that could result in this behavior. It was fixed a few months ago:
https://gitlab.kitware.com/cmake/cmake/-/commit/586d483983a963ad2a73511080f591bc4aea8578
According to their current git history, this hasn't made it into a release yet, but will be part of Cmake 3.23, which is currently out as release candidate 2.
It may be that you'll have to patch your copy of FindOpenSSL.cmake, or build your own Cmake from their current release branch

@levitte This has no effect because if("${OPENSSL_VERSION_MAJOR}.${OPENSSL_VERSION_MAJOR}.${OPENSSL_VERSION_FIX}" VERSION_GREATER_EQUAL "0.9.8") would be true anyway because 3.3.1 is greater than 0.9.8.
The problem was that OPENSSL_VERSION_MAJOR, OPENSSL_VERSION_MINOR and OPENSSL_VERSION_FIX were not defined. It works now!

Ok, so this can be closed now?

Yes, I think so


Items lost in manufacturing

I noticed that when centrafuge upgrades all manufactured items are lost. Dont know if it's true for the rest factories.

Chemical plant ate a whole stack of coal and all plastic that was sitting in the output when it leveled up.

Items that was stored in machine to be used for product, gone if factory level up. Modules were saved (as intended) but ingredients gone. For cheap resources - is not a problem but is a very high problem with expensive products.
For example Centrifuge with Kovarex -> all U235 was cleared after factory was leveled.


az login hangs

Describe the bug
az login hangs.
I used different client machines at different locations (and network connections). They all show the same effect.
During my examinations I noticed that az login uses a German TLD when connecting to login.microsoftonline.de … That domain doesn't answer and that's the issue within az login.
To Reproduce:

use the following Azure CLI config file:
zipped config file (click here to download)
hit az login
wait …

Expected Behavior
az login should display a login screen.
Environment Summary
Windows-10-10.0.19044-SP0
Python 3.10.8
Installer: MSI

azure-cli 2.42.0

Additional Context
az login – Fiddler log

Arbitrary other Microsoft websites are working/responding smoothly

login.microsoftonline.de – tracert

@jiasli for awareness

/ref: #24265

Duplicate of #23775
Nice GIF by the way. 😆


fix: remove all * permissions from role 15

Fix: remove all * permissions from role 15.
This risks being a bit too blunt and some functionality might stop working for some of the removed parts.  In these cases, we'll have to add these parts back one by one without a * permission, and test. So far, this has been mainly tested for cohortdefinition and conceptsets. Anyway, I judged this better than having too broad permissions (most * permissions are likely too broad) and resulting security issues.

@rkboyce please consider merging this to update https://github.com/OHDSI/WebAPI/pull/2342


Decisions on various paths for URL restructure

These routes need to be documented in the URL update spreadsheet

/guides/

Is a Custom Content type; we have 1. http://wellcomecollection.org/guides/YL9OAxIAAB8AHsyv (has an honorary mention in Migrate deprecated content types)

/projects/

Is a Custom Content type; we have 12. http://wellcomecollection.org/projects/YT8ERBAAACMAOBnc

/seasons/

Is a Custom Content type; we have 2. http://wellcomecollection.org/seasons/X84FvhIAACUAqiqp

/concepts/

https://wellcomecollection.org/concepts/fmydsuw2

I would consider /concepts out of scope, but will have a look at the others. Thanks!


Installation Failure Clarification

Hello all,
Could you please provide elaboration on a result your await() function returns?
Using the recommended switch from your "Getting Started" page,
when (val result = packageInstaller.createSession(apkUri).await()) {
        is SessionResult.Success -> println("Success")
        is SessionResult.Error -> println(result.cause.message)
}

we get the following output from result.cause.message
"Session<PHONE_NUMBER> is dead."
What does this mean? Can we resolve this, or is this an issue in the library?
Thank you,
Sam

@samjoshai
Session {id} is dead. install failure is delivered if the OS PackageInstaller session doesn't already exist due to being completed for whatever reason, and when PackageInstallerStatusReceiver, which is a BroadcastReceiver handling install status from the OS, times out. This failure should be quite rare.
To understand whether there's an issue with the library's implementation, please provide answers for the following questions. Do you get this failure frequently? Do you observe a waiting circular loading indicator before getting this result? Does an app get installed in the end, and does it get installed by other means? In which conditions and environment does this failure get delivered (i.e. device, OS version, APK files to be installed compatibility, etc.)?

@solrudev
Thank you for the quick response! Your team is great about responding to developers.
Do you get this failure frequently?

I would say less than 25% of the time, but it happens frequently enough to notice
Do you observe a waiting circular loading indicator before getting this result?
Yes, I briefly observe this.
Does an app get installed in the end, and does it get installed by other means?
The app appears to be successfully installed by the end. This is the only installer being used for the update.
In which conditions and environment does this failure get delivered (i.e. device, OS version, APK files to be installed compatibility, etc.)?
A custom device from an OEM
Android 12
VLC and Firefox which are both A12 compatible
Update is launched from a scheduled timer in a foreground service

@samjoshai

The app appears to be successfully installed by the end.

Are you completely sure? If that's an update, it can be hard to tell, especially when updating to the same version while testing. Ensure that last install timestamp in the system is really updated.

This is the only installer being used for the update.

What I meant with that question is whether you can successfully install the app with other installers.
Can you also provide a sample project where on your end this failure is frequent enough? I didn't encounter it with valid APKs on my devices. This measure was meant to report a failure if PackageInstallerStatusReceiver fails to do so because of some APK parsing errors in the OS ("There was a problem parsing the package").

@solrudev
Is there a debug version of your library that we can use in development?
Perhaps if we use this, then when we observe this issue we can give you better logging. As I stated above, it will be very challenging helping you reproduce since we are doing this in a custom Android OS.

@samjoshai What do you mean by debug version of the library?

@solrudev By debug version of the library, I mean a version where you have verbose logging enabled in your source for example. If that doesn't exist though, no worries.

@samjoshai Hey, did you have any success with investigating the issue?

@solrudev No success. It is clear when this error is reported, the installation was successful though. Unfortunately, I cannot think of a way for you to reproduce without our hardware and OS.

@samjoshai You previously mentioned that you launch sessions from scheduled service. Is it possible that OS on your device doesn't allow BroadcastReceivers to start from background for some reason? For example, with such policy settings as Autostart on Chinese ROMs.

I repeated the installation process according to the documentation from your site.
During the installation process, the installation progress reaches 70% and the user is shown a message that permission must be granted to install programs from third-party sources.
The Android menu opens, where I set the appropriate permission for our application.
After that, I return to the application and for some time observe a waiting circular loading indicator.
After the indicator disappears, the installer session ends with the error "Generic(message=Session {id} is dead.)".
If I start installation again then it is successfully completed.
I used  setRequireUserAction(true) and setConfirmation(Confirmation.IMMEDIATE) because other options lead to installation ending with interruption on 70%.

@gizomo

I used setRequireUserAction(true) and setConfirmation(Confirmation.IMMEDIATE) because other options lead to installation ending with interruption on 70%.

Which errors or misbehavior do you observe with other options?

The Android menu opens, where I set the appropriate permission for our application.
After that, I return to the application and for some time observe a waiting circular loading indicator.
After the indicator disappears, the installer session ends with the error "Generic(message=Session {id} is dead.)".

Does an install confirmation from OS appear after returning?
Also, does install progress always reach 70% specifically when a session is committed, i.e. install confirmation appears?

solrudev

Does an install confirmation from OS appear after returning?
Confirmation window (at fullscreen) appears when installation progress reaches 70%. But there is also a dialog from OS above this confirmation window. This dialog request to grant application permission to install apps from third-party sources. After permission granted the installation confirmation window disappears and we can observe a waiting circular loading indicator for some time. After the indicator disappears, the installer session ends with the error "Generic(message=Session {id} is dead.)".

Also, does install progress always reach 70% specifically when a session is committed, i.e. install confirmation appears?
The install progress always reaches 70%. If we use setConfirmation(Confirmation.DEFERRED) there is no any confirmation window or permission dialog. The installation just hangs out without any errors. If we use setRequireUserAction(false) the installer session ends with the error "Generic(message=Session {id} is dead.)".

@gizomo Hey, if you haven't seen it yet, I replied to your comment in a new issue: #84. I'm waiting for your feedback.

@samjoshai Do you experience the discussed failure with the same rate using latest Ackpine version?


Add generated comments to Kotlin files (default true)

Will respect the existing --omit-file-comments flag
Fixes #485

Codecov Report

Merging #486 (eaf3841) into master (96a7179) will increase coverage by 0.02%.
The diff coverage is 85.71%.

@@             Coverage Diff              @@
##             master     #486      +/-   ##
============================================
+ Coverage     59.15%   59.18%   +0.02%
- Complexity      865      868       +3
============================================
  Files            69       69
  Lines          6481     6488       +7
  Branches       1010     1012       +2
============================================
+ Hits           3834     3840       +6
- Misses         2345     2346       +1
  Partials        302      302

Impacted Files
Coverage Δ

.../com/microsoft/thrifty/compiler/ThriftyCompiler.kt
0.00% <0.00%> (ø)

.../com/microsoft/thrifty/kgen/KotlinCodeGenerator.kt
84.81% <100.00%> (+0.06%)
:arrow_up:

Continue to review full report at Codecov.

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

Sorry for the delay; crazy family visit == no time to think about anything!  I appreciate the effort.  This basically LGTM but I'd like to make the comment an opt-in feature - that is, it should default to false.

@benjamin-bader How dare you prioritize family over your volunteer OSS work! :) Anyway, I responded to your comment re the default value. I can update the docs if we end up having different behavior for Java vs Kotlin.

 :)

It's one weakness that I indulge, I admit.
Thanks for your contributions, as always!


Allow group-mapping to be optional

Currently, the plugin requires LDAP group membership to login.  In our install I have disabled that functionality by commenting out the required code in authLdap.php.
I'd like to see an option to enable/disable the use of groups for assigning roles.

As you mentioned in your email, if using groups for roles is not used, you'll need another option to specify the role to be assigned to new users.

So

EITHER assign a new users role according to the LDAP-group membership (current behaviour)
OR assign all new users to the same given role (new feature)

This has just been implemented with PR #24
You can leave all LDAP-Groups empty and just set a default role which will then be assigned to all users loging in.

Ideally, the LDAP query to search for groups would be "disabled" with this option (why waste a connection to LDAP if the query results will not be used).

OK. That would then be an extra option to that Don't check LDAP-Groups at all and go straight to the default group. Thanks for the input!

This has been implemented by @EmTeedee in #26


[api-docs] Fix: Fix link in stage instance (create|update|delete)

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/71a12debd8a12a6c862bcf8336a6787cfa95a670

This commit will not effect our lib's code. No changes required.


Update OIDC settings after cluster creation

Issue #, if available:
#676
Description of changes:

These changes lets the user update the OIDC settings after cluster creation.

Testing (if applicable):

Unit tests
Manual cluster creation to test that oidc settings were getting updated after changing the spec.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest


Enable pedantic warnings

Last week I learned that -Wall and -Wextra doesn't enable all warnings; there's -Wpedantic too.
When enabled, it had the following warnings (treated as errors):
$ make
In file included from test_c_kzg_4844.c:5:
./tinytest.h:74:56: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes]
   74 | void tt_execute(const char* name, void (*test_function)())
      |                                                        ^
      |                                                         void
test_c_kzg_4844.c:783:25: error: binary integer literals are a GNU extension [-Werror,-Wgnu-binary-literal]
  783 |     uint32_t original = 0b00000000000000000000000000000000;
      |                         ^
test_c_kzg_4844.c:784:25: error: binary integer literals are a GNU extension [-Werror,-Wgnu-binary-literal]
  784 |     uint32_t reversed = 0b00000000000000000000000000000000;
      |                         ^
test_c_kzg_4844.c:789:25: error: binary integer literals are a GNU extension [-Werror,-Wgnu-binary-literal]
  789 |     uint32_t original = 0b10101000011111100000000000000010;
      |                         ^
test_c_kzg_4844.c:790:25: error: binary integer literals are a GNU extension [-Werror,-Wgnu-binary-literal]
  790 |     uint32_t reversed = 0b01000000000000000111111000010101;
      |                         ^
test_c_kzg_4844.c:795:25: error: binary integer literals are a GNU extension [-Werror,-Wgnu-binary-literal]
  795 |     uint32_t original = 0b11111111111111111111111111111111;
      |                         ^
test_c_kzg_4844.c:796:25: error: binary integer literals are a GNU extension [-Werror,-Wgnu-binary-literal]
  796 |     uint32_t reversed = 0b11111111111111111111111111111111;
      |                         ^
7 errors generated.
make: *** [test_c_kzg_4844] Error 1

Added these options to other bindings that compile their own C code too. For the java bindings, I needed to deal with the "unused argument" warnings; create a simple NOT_USED macro for this. These parameters can't be removed.


Swift demo won't build using latest AWS SDK

When using the 2.4.0 SDK the build fails with an error:

/opt/demos/aws-sdk-ios-samples/CognitoSync-Sample/Swift/CognitoSyncDemo/CognitoSyncDemo-Bridging-Header.h:19:9: note: in file included from /opt/demos/aws-sdk-ios-samples/CognitoSync-Sample/Swift/CognitoSyncDemo/CognitoSyncDemo-Bridging-Header.h:19:
import "DeveloperAuthenticatedIdentityProvider.h"
    ^

/opt/demos/aws-sdk-ios-samples/CognitoSync-Sample/Swift/CognitoSyncDemo/DeveloperAuthenticated/DeveloperAuthenticatedIdentityProvider.h:20:53: error: cannot find interface declaration for 'AWSAbstractCognitoIdentityProvider', superclass of 'DeveloperAuthenticatedIdentityProvider'
@ interface DeveloperAuthenticatedIdentityProvider : AWSAbstractCognitoIdentityProvider
^
:0: error: failed to import bridging header '/opt/demos/aws-sdk-ios-samples/CognitoSync-Sample/Swift/CognitoSyncDemo/CognitoSyncDemo-Bridging-Header.h'

Will it be updated to use the latest APIs?

This is linked to this issue:
https://github.com/aws/aws-sdk-ios/issues/357
If you use 2.3.6 it should work, we're waiting for the samples to be updated.

All sample apps are tied to specific versions of the SDK as configured in Podfile. Currently, they don't support 2.4.0. We will update the samples when they are ready. Thanks.

@yosuke-matsuda is there any news? I'm eagerly awaiting for this fix

Any updates on this?

It is now almost August....3 months down the road and STILL no working example!  Has amazon dumped support for Cognito?

Where is support for 2.4.0?

Hi @Marini83,
Have you tried the samples locating in the this branch ?

Swift demo has been update link


Dependency Injection built in to Azure Functions

Microsoft released Dependency Injection functionality recently which is built on top of the ASP.NET Core Dependency Injection Features. What advantages and disadvantages does providing this solution using Attributes provide their functionality does not?
Is one or the other better performing? Is this faster cold boot? Is there functionality missing/provided by the other? What does compatibility look like with different IoC Containers, or package dependencies?
http://aka.ms/functions-di-docs

One thing I've noticed is that this package has an [Inject] attribute which let's you inject the dependency directly into the parameter of the function.
The official DI implementation you have to inject it into a constructor which means that you can't have a static class/function.
Unless I'm missing something, but from that link there wasn't any usage examples so I'm going by: https://github.com/Azure/azure-functions-dotnet-extensions/tree/master/src/samples/DependencyInjection

I just blogged about how to do setup and consume with the new bits and pieces
https://blog.rasmustc.com/azure-functions-dependency-injection/


Hoxy's implementation of cheerio is incompatible with XHTML

Took some digging to figure out why some specific pages were not rendering correctly for me. Turns out it is due to how cheerio is implemented for as:'$' intercepts. The pages were using XML-flavored markup and hoxy always instructs cheerio to render HTML, causing syntax errors which are only detected client-side.
Test case:
Spin up hoxy with an empty intercept like this (does not need to edit the response):
proxy.intercept(
    {
        phase    : 'response',   // run before we send anything back to the client
        as       : '$'           // ask for a cheerio object, to manipulate DOM
    },
    function (req, resp) {
    }
);

Then visit a page with XML, such as:

http://www.skechers.com
http://www.synergycons.com/tutorials/xhtml2.asp (visual alert when it is broken!)

Responsible code:
Line 26 in lib/dom-reader.js:
DomReader.prototype.toString = function(){
  return this._$.html();  // line 26
};

Changing that to...
  return this._$.xml();

...fixes the problem.
But of course this has negative side-effects for non-XML pages. Seems like something that needs some thoughtful design and automated tests. The MIME type listed in HTTP headers often is not correct, so I'm not sure at the moment how to make this reliable without a lot of overhead. It could parse and render an exact copy with both strategies internally and then detect which one caused the least amount of change... ;)

FYI, I have posted a bounty for this issue on Bountysource.
https://www.bountysource.com/issues/6593289-hoxy-s-implementation-of-cheerio-is-incompatible-with-xhtml
My research indicates that this is going to be more problematic.

It looks like we must make the decision about XML vs HTML prior to doing cheerio.load() and then pass xmlMode:true into the .load method, based on that decision.
Even I am incorrect about #1, I have not yet found a way to access the doctype from within cheerio, which would be my preferred way.
If we have to resort to string searching for the doctype, experience has shown me it will be error prone - for example, some pages set the doctype based on conditional comments to make IE behave differently. So searching the string for the correct instance is not easy. Though, we can probably assume they wouldn't switch between HTML vs XHTML like this. So maybe its a moot point.

Nice! Well if someone jumps in with an acceptable fix then I'll certainly review and merge it in. Otherwise I may take a crack at it soon.
Now that you mention it I suspect xmlMode:true should be set for any actual xml mime-type, regardless of doctype, since in that case you'd truly want to parse it as XML (with the caveat that cheerio isn't a conforming parser). Likewise set it false for any non-xml mime type since then we're likely dealing with tag soup and would not want cheerio to treat <script> content as markup, etc.

Take a look at the branch cheerio-xml.
Basically, if the mime type is xml, cheerio will parse with xmlMode:true, otherwise xmlMode:false.
NOTE: if the markup contains an xhtml doctype but is sent with a non-xml mime type, it will be parsed with xmlMode:false. This is because with xmlMode:true, per the htmlparser2 docs, <script> and <style> content would be treated as markup, which would screw up a lot pages.
Then, if the doctype is xhtml OR if the mime type is xml, then it will output xml(), otherwise html(). Doctype is gotten from cheerio, although in a sort of roundabout way. (see https://github.com/cheeriojs/cheerio/issues/640) Also added a few tests in this branch.

(Pasted from a PR thread) It looks like cheerio has fundamental issues with XML.
// test.js
var cheerio = require('cheerio')
var input = '<foo title="foo & bar"/>'
var $ = cheerio.load(input,{xmlMode:true})
var output = $.xml()
console.log(input)
console.log(output)

$ node test.js
<foo title="foo & bar"/>
<foo title="foo & bar"/>

For tag soup pages this shouldn't matter, but sketchers.com uses a client-side XSL transform and errors like the above completely break the page. I don't know if there's a way to force cheerio to conform more strictly, beyond what's shown above. Draconian xml compliance is probably not on their radar.

Thanks for the work on this. Feel free to claim the bounty: https://www.bountysource.com/issues/6593289-hoxy-s-implementation-of-cheerio-is-incompatible-with-xhtml/developers


Conda / Verilator / compiler issue

I created this repo as a minimal testcase for the issue in https://github.com/google/CFU-Playground/issues/568.   Just follow the README.md.
The root issue could be the litex-hub packages, or Conda, or Verilator, or make-env.
Many potential CFU Playground users are hitting this issue.
@PiotrZierhoffer can you find someone to take a look?

When I'm in the Conda env, CC is defined, and it points to the compiler executable that does exist:
$ echo $CC
$TOP_DIR/env/conda/envs/verilator-make-env/bin/x86_64-conda_cos6-linux-gnu-cc

After the recent update by Adam, I no longer see the issue with the compiler not found.
However, there are issues when it comes to linking the Verilated simulation with testbench.   To illustrate, I have added a make link target in the test/ directory Makefile.
When I run this, I get:
...
echo "" > Vadd__ALL.verilator_deplist.tmp
Archive x86_64-conda-linux-gnu-ar -rcs Vadd__ALL.a Vadd__ALL.o
x86_64-conda-linux-gnu-c++ -Wl,-O2 -Wl,--sort-common -Wl,--as-needed -Wl,-z,relro -Wl,-z,now -Wl,--disable-new-dtags -Wl,--gc-sections -Wl,-rpath,/home/tim/tcal-x/verilator-make-env-test/env/conda/envs/verilator-make-env/lib -Wl,-rpath-link,/home/tim/tcal-x/verilator-make-env-test/env/conda/envs/verilator-make-env/lib -L/home/tim/tcal-x/verilator-make-env-test/env/conda/envs/verilator-make-env/lib    tb.o verilated.o Vadd__ALL.a      -o Vadd
/home/tim/tcal-x/verilator-make-env-test/env/conda/envs/verilator-make-env/bin/../lib/gcc/x86_64-conda-linux-gnu/11.2.0/../../../../x86_64-conda-linux-gnu/bin/ld: /home/tim/tcal-x/verilator-make-env-test/env/conda/envs/verilator-make-env/lib/libstdc++.so: undefined reference to<EMAIL_ADDRESS>/home/tim/tcal-x/verilator-make-env-test/env/conda/envs/verilator-make-env/bin/../lib/gcc/x86_64-conda-linux-gnu/11.2.0/../../../../x86_64-conda-linux-gnu/bin/ld: /home/tim/tcal-x/verilator-make-env-test/env/conda/envs/verilator-make-env/lib/libstdc++.so: undefined reference to<EMAIL_ADDRESS>/home/tim/tcal-x/verilator-make-env-test/env/conda/envs/verilator-make-env/bin/../lib/gcc/x86_64-conda-linux-gnu/11.2.0/../../../../x86_64-conda-linux-gnu/bin/ld: /home/tim/tcal-x/verilator-make-env-test/env/conda/envs/verilator-make-env/lib/libstdc++.so: undefined reference to<EMAIL_ADDRESS>collect2: error: ld returned 1 exit status
make[3]: *** [Vadd.mk:65: Vadd] Error 1
rm Vadd__ALL.verilator_deplist.tmp


docs: fix reference to vs code plugin to match repo

The docs refer to an vscode/egglog-1.0.0 or vscode/vscode directory, but neither of these exist: it should be (it seems) vscode/eggsmol-1.0.0.

Thanks! I think we missed this in the great rename.


Add method to RoadModel to get random point

I think this method would be better suited in the Graph interface (and thus it's implementing classes). I propose the following signature:
public Point getRandomNode(RandomGenerator rnd);
I think getRandomNode is preferred over getRandomPoint since it better describes what it does (it will never return a point which is not a node).

I agree that the real logic should be in Graph, but if you look at
RoadModel, it currently contains a lot of similar methods: getDistance,
getShortestPath, ...
Also, all these methods use the type 'Point'.
From a user perspective it is probably more intuitive that RoadModel
would then contain a method getRandomPoint (which could use a method
from graph).
Finally, does a developer need access to the underlying graph of the
roadmodel?
On 23/02/12 22:23, Rinde van Lon wrote:

I think this method would be better suited in the Graph interface (and thus it's implementing classes). I propose the following signature:
public Point getRandomNode(RandomGenerator rnd);
I think getRandomNode is preferred over getRandomPoint since it better describes what it does (it will never return a point which is not a node).

Reply to this email directly or view it on GitHub:
https://github.com/rinde/RinSim/issues/8#issuecomment-4145578

Ideally, most of the graph related methods should be removed from the RoadModel. I think the only graph related method that should remain is getGraph() which returns an unmodifiable graph reference (and addGraph() as well). The reason that I want to move all graph related methods is that right now there is some duplication in responsibilities, graph related stuff should be in a graph, things on top of a graph should be in the RoadModel.
Finally, does a developer need access to the underlying graph of the
roadmodel?
In my experience, yes. I always needed read access to the graph. But I agree that maybe we can hide this behind an API.


Allow get_param_values() to work on class (fixes #162).

Coverage increased (+0.02%) to 74.471% when pulling 20a806b7e8014a46350eee006f11c495e81b6f63 on issue162-get_param_values_on_class into e2da17613899974323f68667689c4dfddf0143b1 on master.

Looks good, thanks.


Implement Admin Landing Page

Please do work for this task in a branch named issue-24.

Need to update tests.testcafe.js to expect the correct page navigation as well, so it can pass continuous integration


Switch tests and examples to vtgate discoverygateway.

@guoliang100

It looks like a lot of tests are still making assumptions about the specific way shardgateway works. I'll take a look at them tomorrow.

@guoliang100 I had to update some tests to work with discoverygateway. Can you take another look? Thanks.

LGTM