Report aggregate activity data to influxdb

Purpose: to see how/whether users are using the features
Information I think would be interesting are:

joined pickup
left pickup
did pickup (at least, had >0 collectors at time of pickup)
missed pickup (had 0 collectors at time of pickup)
wrote mesage on group wall
joined group
left group
gave feedback
visited page associated with group (using new mark_user_activity)
member count
active member count (last activity <1 week|day|30day|year ago)
store count
new version deployed

Should not be personally identifiable as we are supportive of data privacy, so only include group id as the identifiable information.
Most of them would be sent when the event occurs. Counts would be sent via periodic huey task, once per day would be sufficient I think.
@tiltec are you comfortable/uncomfortable with this amount of data collection?

Seems good, as it would give us quite deep insight what people do on karrot.world. I don't have concerns, as the activity of a single user would most probably not identifiable through this data. Except if a group only has few members, but that seems tolerable, or we could enable activity reporting only when a group has more than 5 members.
This feature should also give us insight in what data we want to show in the website -> #355

In general this looks good.
I wonder what questions you might envisage answering with these data. Thinking a little about this can be helpful in focussing data collection and revealing something you might be able to do.
Naïvely, I can write some suggestion questions but those closer to the site will have much better ideas...

Is the user base growing?
Where is the site being used the most?
Are current users increasing their usage of the site?
Are there different types of users?
Can we say something about differences between people who leave / stay?
If someone joins their first pickup - how likely is it that they'll join a second?

Next step would be to create Grafana dashboards, possibly in version control with https://github.com/weaveworks/grafanalib

I had a play with grafanalib, and https://github.com/grafana/grafonnet-lib, but neither had good support for influxdb, then I started building a nodejs builder (which is well suited to building up json data structures), but then the task was bigger than I felt doing at that time.

Seems mostly done. We could add the dashboard JSON export to karrot-backend git or yuca ansible git from time to time, in order to not lose configuration on bad luck.


Dart 1.4 unable to open package files, import 'package:' fails

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?

Download latest 1.4 release
Open sample project dartiverse_search
Try to run

What is the expected output? What do you see instead?
Expected import 'package:http_server/http_server.dart' as http_server to resolve file path.
Instead when attempt to run example:
C:\dart\dart-sdk\bin\dart.exe --enable-checked-mode --debug:51675 server.dart
Unable to open file: C:\dart\samples\dartiverse_search\bin\packages\http_server\http_server.dart'file:///C:/dart/samples/dartiverse_search/bin/server.dart': error: line 11 pos 1: library handler failed
import 'package:http_server/http_server.dart' as http_server;
What version of the product are you using? On what operating system?
Dart 1.4.0, Windows 8.1

This issue has been moved to dart-lang/pub#985.


Consistency fixes for XML docs

"Requests *" instead of "Request may still be in flight." (there could be multiple ones if I understand it correctly)
adds missing . at the end of comments
removes empty <returns></returns> blocks
removes "This must be settable!" comments in IHostingEnvironment. This should be self-explanatory since the property is get;set; and the xml doc contains "Gets or sets"

Hi @cwe1ss, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


[Internal] JSON Binary Encoding: Adds support for writing unsigned 64-bit integer values

Description
This PR adds support for serializing unsigned Int64 values in Binary encoding, ensuring that values exceeding the maximum signed Int64 limit are preserved without conversion to floating-point doubles. This enhancement enables full-fidelity roundtrips between Text and Binary encoding formats. For values outside the signed Int64 range, retrieval from either the JSON Reader or Navigator (across all encoding formats) will return them as floating-point doubles.
Type of change

[ ] New feature (non-breaking change which adds functionality)

Closing issues
None

Does this change contain any bugfixes (either by themselves or for previous commit)? Asking to see if it should be included in the release that's in progress.

It mainly adds support of reading/writing UInt64 values, which is soon to be needed once writing such values is enabled in the backend.

In reply to:<PHONE_NUMBER>


How to Access The Numpy Version of the Image

After the inference is done on the image, what's a good way of accessing the numpy version of the image ( since NiftyNet already converts ) and save it ?

Hello, are you Chinese? Do you have time to tell me how to enter any size image?

Hi @yufeihun it's in NIfTI format, one option would be using NiBabel (http://nipy.org/nibabel/nibabel_images.html#getting-the-image-data-the-easy-way):
import nibabel
numpy_array = nibabel.load_data('image.nii.gz').get_data()


ExGetFirmwareEnvironmentVariable / "remarks" should not recommend the zero guid as "dummy guid"

[Enter feedback here]
The Remarks section for ExGetFirmwareEnvironmentVariable recommends,

To identify these conditions, call the function with a dummy VariableName
string (for example, an empty string) and a dummy GUID such as "{00000000-0000-0000-0000-000000000000}" for the VendorGuid parameter.

This is wrong; the reader should be instructed to generate their own GUID with sufficient randomness (such as the uuidgen utility), statically of course, and then look for whatever variable in that namespace.
Alternatively, Microsoft should invent a well-known GUID (also with uuidgen), or introduce another variable under a namespace GUID Microsoft already owns, and then normatively specify that (GUID, variable) pair for the check.
The zero GUID is a bad choice because

it was not generated with any amount of randomness, and
it is not clearly associated with any one vendor (in any specification).

Therefore there's a high risk of conflicts between variable names within that namespace.
Thanks.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ebf9376e-f0d1-fe4b-8be4-82ab23718d5b
Version Independent ID: a0eedd57-48a4-44b7-1c5d-22c69ace3482
Content: ExGetFirmwareEnvironmentVariable function - Windows drivers
Content Source: wdk-ddi-src/content/wdm/nf-wdm-exgetfirmwareenvironmentvariable.md
Product: windows-hardware
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Thanks for the feedback @lersek, I just made this change, should be live within the hour.  Apologies for the delay.  Please note that you can also click Edit from the original documentation page if you'd like to submit a PR against the docs..


Ubuntu 16.04 server start issue.

As title, I reinstalled the bot completely and still got this error.
"
Traceback (most recent call last):
File "pokecli.py", line 843, in
main()
File "pokecli.py", line 191, in main
bot = start_bot(bot, config)
File "pokecli.py", line 145, in start_bot
bot.start()
File "/home/name/PokemonGo-Bot/pokemongo_bot/init.py", line 144, in start
self._setup_api()
File "/home/name/PokemonGo-Bot/pokemongo_bot/init.py", line 983, in _setup_api
self.login()
File "/home/name/PokemonGo-Bot/pokemongo_bot/init.py", line 917, in login
str(self.config.password))
File "/home/name/PokemonGo-Bot/pokemongo_bot/api_wrapper.py", line 98, in login
password=password
File "/home/name/PokemonGo-Bot/src/pgoapi/pgoapi/pgoapi.py", line 94, in set_authentication
if not self._auth_provider.user_login(username, password):
File "/home/name/PokemonGo-Bot/src/pgoapi/pgoapi/auth_google.py", line 58, in user_login
user_login = perform_master_login(username, password, self.GOOGLE_LOGIN_ANDROID_ID, proxy=self._proxy)
AttributeError: AuthGoogle instance has no attribute '_proxy'
[2016-09-23 20:15:09] [MainThread] [sentry.errors] [ERROR] Sentry responded with an error: 'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128) (url: https://app.getsentry.com/api/90254/store/)
Traceback (most recent call last):
File "/home/name/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/transport/threaded.py", line 174, in send_sync
super(ThreadedHTTPTransport, self).send(data, headers)
File "/home/name/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/transport/http.py", line 47, in send
ca_certs=self.ca_certs,
File "/home/name/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/utils/http.py", line 66, in urlopen
return opener.open(url, data, timeout)
File "/home/name/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 494, in open
response = self._open(req, data)
File "/home/name/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 512, in _open
'_open', req)
File "/home/name/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 466, in _call_chain
result = func(*args)
File "/home/name/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/utils/http.py", line 46, in https_open
return self.do_open(ValidHTTPSConnection, req)
File "/home/name/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 1284, in do_open
h.request(req.get_method(), req.selector, req.data, headers)
File "/usr/lib/python2.7/httplib.py", line 1057, in request
self._send_request(method, url, body, headers)
File "/usr/lib/python2.7/httplib.py", line 1097, in _send_request
self.endheaders(body)
File "/usr/lib/python2.7/httplib.py", line 1053, in endheaders
self._send_output(message_body)
File "/usr/lib/python2.7/httplib.py", line 895, in _send_output
msg += message_body
UnicodeDecodeError: 'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128)
[2016-09-23 20:15:09] [MainThread] [sentry.errors.uncaught] [ERROR] [u"AttributeError: AuthGoogle instance has no attribute '_proxy'", u'  File "pokecli.py", line 843, in ', u'  File "pokecli.py", line 191, in main', u'  File "pokecli.py", line 145, in start_bot', u'  File "pokemongo_bot/init.py", line 144, in start', u'  File "pokemongo_bot/init.py", line 983, in _setup_api', u'  File "pokemongo_bot/init.py", line 917, in login', u'  File "pokemongo_bot/api_wrapper.py", line 98, in login', u'  File "pgoapi/pgoapi.py", line 94, in set_authentication', u'  File "pgoapi/auth_google.py", line 58, in user_login']
Fri Sep 23 20:15:09 EDT 2016 Pokebot  Stopped.
Press any button or wait 20 seconds to continue.
"

+1

This is a known issue with the pgoapi, but should be fixed in latest dev. Update to latest version and make sure you run pip install -r requirements.txt --upgrade

Sorry, updated and retried, still fails :(

Reopening then. Maybe there's a new issue.

I use Ubuntu as well but I haven't this problem... Maybe if you copy whole logfile since "./run.sh" I can help for more troouble.

You are running dev branch, yes?

nope, I am running main branch only. (I don't know how to move to dev branch. :D I am new for git command.)
Just latest update have problem with proxy but modify same as an advise in thread so it fixed.

Ah, well in that case... PR is ready to merge fix into master. #5645. Once that's done, this issue should be fixed.

Yes, it is fixed now, thanks! :)


Minimized, the voice activation

If the app is minimized, other members of the call don't heare me speaking. (I think it is the voice acitvation  that has a problem)

Are you using MobileDiscord on Windows 10 desktop?

Yes exactyl. MobileDiscord uses less space.


Android SDK got response code 403

Done

Hi @pqminh!
403 probably means a mismatch in the dev key or app_id (when using channel it is considered to be part of the app_id in AppsFlyer). After a quick check, I am not able to find an app in your account that would use channel "productionabc" that you have specified in the Android Manifest.
Please, open a ticket to our support team and they will assist you in setting up "out of store" channel properly
Thank you!

Sorry, I has contact support team and issue was resolved

I also have a 403 error. Tech support is not responding. How can I find a solution?

Sorry, I has contact support team and issue was resolved

How did you solve the issue?

i solve, u need upgrade Zero plan to paid for mobile


fix (DocsBuild):  Ensure graceful-fs resolves to a version compatible with node12  - fixes #7271

Pin version of graceful-fs so that documentation can build on node12+ without error.

I'm putting a hold on this one; the leafdoc update from #7263 should already solve the issues with graceful-fs.

@IvanSanchez, checked our the #7263 branch, works fine here, so I'll close this.


ibm-object-storage-plugin - update chart version and form view support

@pedjak @morstad I have a raised a PR from my fork and the Initial PR scan seems to be failing. Earlier when I raised the PR, I did not hit with this issue.
This seems to be an issue with github actions -https://github.community/t/github-actions-are-severely-limited-on-prs/18179#M9249
Could you please suggest on how to go about this ?

Tested and verified my changes on OCP 4.6 cluster.

can you please squash the commits?

@pedjak I have squashed the commits into single commit. Could you let me know if any more review comments / changes required for this PR to be merged..


enableFlutterDriverExtension() in main.dart stops keyboard, if i remove it appium-flutter-driver is showing handshaking issue

@GarimaVats
EnableFlutterDriverExtention is mandatory for the automation.
Please reach the dev team it may be due to dart version compatibility issue.

Can you give an example of working versions?

v1.12.13+hotfix.7 or v1.9.1+hotfix.6 should both work. If it's not working for you, can you report the device and OS you are using?

@truongsinh it seems it's already reported in flutter community nothing to do in Appium flutter driver. I have tried with latest dart and flutter version.
https://github.com/flutter/flutter/issues/15415

@truongsinh I tried almost all appium enter key function. I was using appium 1.16.0. I used it for both android and iOS devices and both OS Mac and windows.

@truongsinh I tried almost all appium enter key function. I was using appium 1.16.0. I used it for both android and iOS devices and both OS Mac and windows.

From https://github.com/flutter/flutter/issues/15415#issuecomment-581532972, looks like @jonahwilliams meant we have to have the solution ourselves. @GarimaVats can you describe the reason you need to interact with on-screen keyboard?

@truongsinh
it blocks us to use sendKeys() function on android/iOS native.
There must be a scenario where we have to use sendKeys() to handle any native element in android or iOS (e.g XCUIElementTypePickerWheel)

So let me see if I understand you correctly, here's the reproducing step:

open app
switch to Flutter context
select text box
send some keystrokes
switch to native context
send some keystrokes

It is at the 6th step that you are blocked?

@truongsinh
Yes, it is throwing an exception on 6th step.

@truongsinh looks like @jonahwilliams added caveats for this.
https://github.com/flutter/flutter/pull/50017.
Whether we need to add it in our README file as well.

@truongsinh
Hello,
Do you have any news please?

Do you have any news please? ?

Respected @shibupanda any update for the enableFlutterDriverExtension() in main.dart and stops keyboard and unable to enter any text into textfield

has anyone found a solutions to this issue yet?

has anyone found a solution for this yet?

Any solution on this ?

The keyboard itself does not appear as the flutter_driver lib behavior.
So far, sending keys via uia2/xcuitest native context could be a workaround as this driver https://github.com/appium/appium-flutter-driver/issues/607#issuecomment-1774184865


Use ddt for test_transpile_across_optimization_levels

Summary
This commit updates the test_transpile_across_optimization_levels in the
TestParameters test class to leverage ddt instead of a for loop for
testing each optimization level. Right now we're hitting sporadic
failures in CI related to that test case and the only candidate for
causing a performance issue is only applicable to optimization level 3.
To get better data about the failure mode it is important to know which
optimization level a failure is occuring at. By using ddt and
parameterizing the test over each optimization level, each opt level
becomes a separate test method with a unique test id (which identifies
the optimization level) so we can easily verify if the failures only
occur at level 3 or not.
Details and comments

Removing automerge to prioritize commits critical for the release in the merge queue. After they have all merged we can retag it


[boost-modular-build-helper]Fix build fail due to missing library file

Describe the pull request

What does your PR fix?
Fixes incorrect library suffixes set for MinGW builds of boost modules (e.g. boost-thread)

Which triplets are supported/not supported? Have you updated the CI baseline?
This will not affect any other triplets except mingw, CI baseline is not updated.

Does your PR follow the maintainer guide?
I have bumped a port version

Thanks for the fix!


[5.3] Add a method to assert type of values in a JSON structure

Hi,
I'm currently building a REST API, and I would like to assert that my endpoints return the right JSON using seeJsonStructure. Classic case. But, the point is that I use Faker to generate fixtures, and naturally my tests are broken after each db:seed. Of course I can use seeJson or seeJsonEquals, but IMHO we miss another method to assert the structure of the JSON and types of values (basically integer, string, boolean, array) as:
$this->seeJsonTypedStructure([
  'id' => 'integer',
  'name' => 'string',
  'isActive' => 'boolean',
  'roles' => 'array',
]);

It's why I've started to work on seeJsonTypedStructure method here : https://github.com/welcoMattic/framework/commit/992a4c4524f0eddea0c3af64af26a2e32a9e3d94
It's open to discuss about the real value of this method ;)
Thanks

It could be nice to valide a JSON when the data are generated by faker ! Good idea !

Nice idea. Since we are talking about JSON I think it should cover all the JSON data types?

Number
String
Boolean
Array
Object
null

@lasselehtinen Yes, I'm agree with that. I'll improve my code before opening a PR here.

I like the idea and suggest that you open a PR with the suggested changes.
Closing the issue since it's not a bug report, let's continue the discussion on a PR or if you'd like you can create an issue in the internals repository.
https://github.com/laravel/internals

@themsaid Ok thank you, I'll open an issue in internals ;)
BTW, congrats for your job ! 🎉

Thank you! 😁


Migrate from travis-ci.org to .com

The .org site will be read-only starting 2020-12-31.
First step: sign the Toblerity org up for the migration beta.

@rbuffat any objections to me moving the project over tonight?

I wasn't even aware that there are any changes to travis org.

I was also unaware. Then I saw this 2 days ago<EMAIL_ADDRESS>

Done! https://travis-ci.com/github/Toblerity/Fiona


istio.io is down

Bug Description

reposting here for visibility (original issue)
Version
site

Additional Information
No response

As far as we know this was a netlify outage that is resolved and we cannot control, so I think there is no action here. Thanks for reporting!


Shader error

I am getting a error here:
Traceback (most recent call last):
File "C:\Users\xingh\Downloads\python-minecraft-clone-master\python-minecraft-clone-master\episode-3\main.py", line 87, in
game = Game()
File "C:\Users\xingh\Downloads\python-minecraft-clone-master\python-minecraft-clone-master\episode-3\main.py", line 81, in init
self.window = Window(config = self.config, width = 800, height = 600, caption = "Minecraft clone", resizable = True, vsync = False)
File "C:\Users\xingh\Downloads\python-minecraft-clone-master\python-minecraft-clone-master\episode-3\main.py", line 61, in init
self.shader = shader.Shader("vert.glsl", "frag.glsl")
File "C:\Users\xingh\Downloads\python-minecraft-clone-master\python-minecraft-clone-master\episode-3\shader.py", line 46, in init
create_shader(self.vert_shader, vert_path)
File "C:\Users\xingh\Downloads\python-minecraft-clone-master\python-minecraft-clone-master\episode-3\shader.py", line 37, in create_shader
raise Shader_error(str(log_buffer.value))
shader.Shader_error: b''

Thank you. Could you provide the vendor and model of your CPU? This is a problem a few people have been having and I haven't yet been able to reproduce it.
In the meantime, you can simply remove the error raising code.

I was using a intel Core i5 vpro 2.3 ghz.

Ok, does it work without the error handling code for you or not? I'll try to sort out that bug asap

I think I had the error earlier and I made a temporary fix for it by deleting the raise line for the error. The permanent fix would be replacing that with:
if log_length and log_buffer.value: 		raise Shader_error(str(log_buffer.value))

https://github.com/obiwac/python-minecraft-clone/pull/25

Ok, when log_buffer.value evaluated to false, what's the value of log_length?

I think it might be c_int(-1) but I need to check.

Then it may be smarter to instead check if log_length > 0.

I will try that.

It outputs c_long(1)

#26

All is fixed! Thanks @drakeerv mate!


The last stage of environment creation using Wordpress blueprint

The last step of environment creation was stuck. The case should be reproduced and investigated.
Steps to Reproduce

Create Peer
Create environment on the peer using Wordpress
Check up the steps of creation

This issue is not supposed to be here. I am closing it in order to open it in proper repository.


Proposed Datasets Actions Not Fully Migrated

Proposed Datasets Actions Not Fully Migrated https://app.clickup.com/t/8688wdt1n

Approved? I can comment but not approve


:fix: NameResolver now has the feature to get status of named resource

[BOT]
E2E results

[BOT]
E2E results

[BOT]
E2E results


Assess references that have been received

Trello
Context and changes
As an assessor that is looking at the reference that have been received about an applicants work history
I need to be able to view the information that a referee has submitted
so that I can  decide whether to award or decline an application
Adds a verify reference requests and review reference request pages.
I'm not 100% sure what the happy path from verifying all references should be so for the moment this PR takes the user to the application overview. This probably isn't right.
Verifying reference requests

Reviewing a reference request

This has been product reviewed so I'll merge this in.


Tests fail because git doesn't know file permissions

Tests fail in my checkout wherever files are asserted to have other permissions than 755 or 644. See http://stackoverflow.com/questions/14557106/git-changes-my-file-permissions-upon-checkout#answer-14557151 for an explanation of Git and file permissions.
Maybe the files under test can be changed to have either of the two permissions known by git? Otherwise, perhaps a Rake task is needed to chmod the test files?

FYI the tests pass after calling:
chmod 664 test/gemfiles/lorem-0.0.1/lib/lorem.rb
chmod 664 test/gemfiles/lorem-0.0.3/bin/lorem
chmod 775 test/gemfiles/lorem-0.0.4/bin/lorem
chmod 664 test/gemfiles/lorem-0.0.4/lib/lorem.rb

Yes, it's unfortunate. We can probably accept both as you suggest.

#29 fixes this


Browse unscanned media files

Browsing via local:directory:path URIs currently shows all directories under local/media_dir, but returns only files that have been imported via mopidy local scan.
While this is useful for checking for unscanned files, it may be confusing for new users. It may be preferrable to return all files, except those matching local/excluded_file_extensions, with local refs for files already in the database, and plain file refs otherwise. Thus, it would be possible to browse (via "Folders") and play unscanned media files, though searching and browsing via "Artists", "Albums", etc. would still be limited to local files already scanned.

Maybe more consistent to only return file refs, with names set to file names.


Fix address search

Actual behavior
The API endpoint /search does not work with an address.
Expected behavior
The API endpoint /search does works with an address.
Steps to reproduce
Request https://betanet-service.lisk.io/api/v1/search?q=6037832807575997134L
Which version(s) does this affect? (Environment, OS, etc...)
0.2.0-beta.1

This bug does not exists anymore.


[crypto] Check both backend features are not enabled simultaneously

This would typically happen with somebody compiling with --features "vanilla" rather than --features "vanilla" --no-default-features.
This is a follow-up to #4970, which made a feature control how we load dalek dependencies, and #5126 and following, which enable the new resolver and therefore help features have real mutually-exclusive semantics.

isn't this going not to work due to our use of --all-features in scripts?

It's my understanding #5126 has obviated the need for --all-features and why I added @metajack in review. Note this passed CI.

@bors-libra r+

:pushpin: Commit d93316e has been approved by metajack

:hourglass: Testing commit d93316e70ef62543abe8ff011926bb47798b5400 with merge 17d9442924c0291cb086e30c92f63f548065cc5e...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: metajack
Pushing 17d9442924c0291cb086e30c92f63f548065cc5e to master...


chore(master): release 0.44.5

:robot: I have created a release beep boop
0.44.5 (2024-08-12)
Bug Fixes

update cheerio to 1.0.0, closes #444 (0d70f63)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/easyops-cn/docusaurus-search-local/releases/tag/v0.44.5 :sunflower:


Add an organizations page

#31, but with scope reduced to simply reading a list of organizations directly from the officers table.
It could be nice to add summary statistics to each organization's page - e.g., an IA score distribution, median/mean employee income, etc.

Merged and deployed to prod! Sorry for the delay, this totally slipped my mind.


Documentation update needed

It seems that the TDVT tests suite (code checked out on Nov 12th, 2019) have changed significantly enough that the documentation on how to run it is not up-to-date anymore (compared to the code from Oct 8th, 2019).
The steps that don't match the new version are not few, but for one tdvt --add_ds and other similar commands don't work anymore.
From what I could tell tdvt --add_ds is now tdvt action --add_ds and tdvt --run is now tdvt run.
py -3 -m tdvt.tdvt --setup doesn't work anymore, and same goes for tdvt --generate. It's a lot of guessing now, which makes tdvt usage time consuming and not user friendly.

Hi, Andrei, thanks for opening the issue. Apologies for the docs issue, we will get them updated as soon as possible.
In the meantime, if you'd like to check out to 99159b8, that release of the sdk should work as expected with the current docs.

@lukewrites thanks for the reply and acknowledgement.
Indeed, the docs work fine for 2019.3, 2019.2 and 2019.1, but there are some fixes (and this one) that I'm eagerly waiting to check out, thus my interest in the 2019.4 version and the up-to-date TDVT suite.

@astefan docs have been updated.


How to render more than one canvas in a single page ?

Sorry for the stupid question , may i ask how to apply 2 canvas, currently the code below is append the canvas to #matterJs , is there a way I can multiple append to #matterJs2 , #matterJs3 ?  Thanks
Render.create({
element: document.getElementById('matterJS'),
});

Easiest thing to do is to probably just create a renderer for each canvas I think, otherwise make a fork of Matter.Render and adapt it as you need.


Creating a pedagogical python notebook

Done with three tasks!

Thanks @Qudirah! Can you split this PR into 2? This PR adds all the files at once.
It's quicker to go through these changes one PR at a time as different tasks take different amount of time in reviewing.


I need to make some additional settings when packaging. My settings cannot be used. Please help me. Thank you.

(venv) lilinze@lilinzedeMac-mini winwinwin % sudo /Users/lilinze/Downloads/winwinwin/venv/bin/python /Users/lilinze/Downloads/winwinwin/qianming.py
Password:
2023-11-18 21:31:55,329 pymacapp >>> appfactory.py @ 49 in .init(...) [DEBUG]: App(self._name='yunwuai') created
2023-11-18 21:31:55,329 pymacapp >>> validators.py @ 27 in .validate_app_name(...) [INFO]: validated: yunwuai
2023-11-18 21:31:55,329 pymacapp >>> pyinstaller.py @ 61 in .spec(...) [DEBUG]:  --name 'yunwuai'
2023-11-18 21:31:55,329 pymacapp >>> validators.py @ 59 in .validate_file(...) [WARNING]: invalid file: /Users/lilinze/Downloads/winwinwin/独立exe启动器1.spec
Traceback (most recent call last):
File "/Users/lilinze/Downloads/winwinwin/qianming.py", line 27, in
app.config(main).build().sign(APP_HASH)
^^^^^^^^^^^^^^^^
File "/Users/lilinze/Downloads/winwinwin/venv/lib/python3.11/site-packages/pymacapp/buildtools/app/appfactory.py", line 88, in config
self._spec = spec(name=self._name,
^^^^^^^^^^^^^^^^^^^^^
File "/Users/lilinze/Downloads/winwinwin/venv/lib/python3.11/site-packages/pymacapp/pyinstaller.py", line 65, in spec
raise BuildException(f"unable to validate {main_script=}")
pymacapp.exceptions.BuildException: unable to validate main_script='/Users/lilinze/Downloads/winwinwin/独立exe启动器1.spec'

Hi @a37836323
The BuildException you are seeing (invalid file: /Users/lilinze/Downloads/winwinwin/独立exe启动器1.spec) is because the file was not found, i.e. the pathname is incorrect. This would solely be because MacOS was unable to locate the file. Please confirm the filepath is correct.
Try running this command in a terminal:
cat /Users/lilinze/Downloads/winwinwin/独立exe启动器1.spec

If you get an error here, then the filepath is definitely wrong; if the above works, then you may want to rename your .spec file to something extremely simple (such as app.spec) and try again. It could be a character limitation issue.


[Multiselect] Multiselect label and tag get overlapped on RTL mode

Steps:

Open the Storybook
Open Multiselect component
Click on RTL tab and click on RTL toggle
Select some items in the dropdown
Actual Result: Label and tag get overlapped

:tada: This issue has been resolved in version 2.40.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


add dpo code.

Type of Change
API not changed
Description
add Direct Preference Optimization algorithm

Can we enable under neuralchat API?

Coverage drops. please add UT to improve.


Fix naming for Westend

It is great to have a hosted version, thanks for deploying that @hamidra.
The Westend version however has a couple of naming issues. It contains also a confusin mix of Polkadot and KSM.
I am pasting a few screenshots below, those issues are likely all linked.
UI

Email


etcd 2.0.0

Hi, when will this changes be released?

@sawanoboly i want to do some more test on the https based calls.  do you need this urgent?

No, I don't need this very urgently.
I will update my etcd cluster to 2.0.0 in next month. And I would like to use API2 via ruby at the same time.
If it's OK with you, shall I help with writing test?

@sawanoboly that will be awesome :-)

Any updates on this @ranjib pretty please ? :)


Use friendsofphp as the vendor in composer

Use "friendsofphp" as the vendor instead of "fabpot" in composer installs.

Travis is failing


Barbarians Try to Attack Your Units and Destroy Your Roads and Cities

Instead of wandering around aimlessly, barbarians should seek loot, from your and other AI civilizations.
They should walk towards your cities, and generally try to be as much as a nuisance as possible.
This could be with individual units, or with small groups of them.  I like the idea of having it vary a bit by tribe - maybe this tribe just attacks Rambo style, but that other tribe attacks less frequently but with some coordination.  Kind of open ended, but more interesting than "beeline to nearest city" is encouraged.  If they find the city is defended, but some Iron is available to pillage?  I'd take the easy target if I were a barbarian.

This one has a PR, there were a lot of things to address on the PR and that's when my availability went down and I never got around to finishing the requested changes on the PR.  So it's still open.


[REQUEST] slim-alpine

I saw that there are already multiple images. But since slim is also available for debian etc, I would love to test it on alpine, since I use alpine.
I ATM use:
FROM python:3-alpine
and would like to test:
FROM python:3-slim-alpine
if available.
So it would be cool to see slim releases being created with the other distros as well:

alpine
debian (already available)
windows

BTW: python 3.10.7 would be nice as well, since somehow it did not go through.

The Alpine-based images are already built in the slim way; they can't really be any smaller and still run python3.

BTW: python 3.10.7 would be nice as well, since somehow it did not go through.

https://github.com/docker-library/python#see-a-change-merged-here-that-doesnt-show-up-on-docker-hub-yet  the PR is already open: https://github.com/docker-library/official-images/pull/13108.

True, sorry - my fault.
Just checked:
$ python3.9 --version
sh: python3.9: not found

Thanks!


[ci] use default CPU/Memory

I've checked CPU charts for a few build of ponylang/ponyc repo and it seems the tasks are not using all the cores and memory.
Having less CPU request will help to run all tasks faster. Right now you run just 2 at a time and will be able to run 8 at a time.

We need that memory and CPU to build LLVM whenever we bust the cache and can't use the cached version.
If there were a way to allocate memory and CPU after we knew if the cache was good, then this change might be possible.
However, if we make this change, when we do bust the cache, it will take forever to build LLVM.


Improved UI

Also resolve the conflicts and squash.

Should be good now. Some testing might be needed for OIDC


MIMIR-601 improve publish job

MIMIR-601

create publish job task (prep publish dataset)
create publish dataset task (publish dataset)
create cleanup publish dataset (delete old, log etc)

On hold as the new lib is very buggy. Waiting for a fix.

Lukker denne siden mye av koden er i master bak feature-toggle


fix(parser): Updated the field_parser regex for pytest-splunk-addon

fix(parser): Updated the field_parser regex for pytest-splunk-addon
Updated the parser from [aA][sS]\s* to [aA][sS]\s+ to parse prefixes using as or AS correctly.
Also added new tests to verify the fixUpdated the field_parser regex for pytest-splunk-addon from [aA][sS]\s* to [aA][sS]\s+ to parse prefixes using as or AS correctly.

Can you please review this fix with IBM-WAS configurations https://github.com/splunk/splunk-add-on-for-ibm-websphere-application-server/blob/master/package/default/props.conf#L336

Please update the target branch from main to develop

Please update the PR with commit conventions mentioned here https://www.conventionalcommits.org/en/v1.0.0/  so as to bypass the CLABot https://github.com/splunk/pytest-splunk-addon/pull/216/checks?check_run_id=1565045557

:tada: This PR is included in version 1.3.14-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.6.0-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


CONNECTION_FAILED

With a builded app on iOS I keep receiving the case CONNECTION_FAILED builded by Phonegap Build cli 6.3.0. It works fine on Android but not on iOS. Anything I'm missing?

Issue is resolved. The URl that I was checking had an infinite redirect, causing the time out.


nan information

dnest4.py:175: RuntimeWarning: invalid value encountered in multiply
  H_estimates[z] = -logz_estimates[z] + np.sum(P_samples[:,z]*logl)
log(Z) = -106.72234101 +- 0.0
Information = nan +- nan nats.

This is happening in the BL2009 example

This happens because some of the log-likelihoods are huge (negative) values and the multiplication underflows.
I traced it back to the log_likelihood function, which outputs -1e300 when logL is infinite of nan

Maybe @eggplantbren can help here, I'm not convinced that the sampling is going well on this example

I'm getting a sensible value for the information. However, some other aspects of the run look unhealthy to me. I'll do some more investigation.

This seems to be solved by the pull request so closing


Create component to display sponsors

Can you recreate Stanford's sponsor listing? It's a grid layout of sponsors and upon hover, there is some additional information about each company.

Add capability to automatically resize the tile upon render based on the tier the sponsor is (Platinum, Gold, Silver, Bronze). This should make the tiles appear either bigger or smaller. The higher tiers should get their own row and the lower tiers should be grouped together and rendered on the same row.

@Zahraa222 assigning this to @sp-95 instead.


[<IP_ADDRESS>-Snapshot] iOS10 no correct display of paper components (all browsers on iOS)

Hallo Community,
Problem: The library doesn't work with all browsers on iOS10. It neither works with version <IP_ADDRESS> nor with version <IP_ADDRESS>-SNAPSHOT.
Googled Information: It's the same problem which is here described: https://www.bountysource.com/issues/37936276-gwt-polymer-framework-does-not-work-on-ios-10-any-browser
Questions: Is it possible to use the latest webcomponents version? Does anyone know how to solve this problem?
Thanks in Advance!

What specific issue are you facing. We use v<IP_ADDRESS> and it works fine on iOS10.
The sample code link you have posted gives 404. Can you rectify it?

Tried demo app compiled with <IP_ADDRESS> and works as expected. Here you have a screenshot.
http://g.recordit.co/9iM2ehQo0m.gif
Feel free to reopen the issue if you provide some code or hints to reproduce the problem.


[WIP] Limiting graph updates to releaseNamespace

I noticed that the origin/release-upgrade/graph is full of upgrades from ocp.  This is because the prowjob informer is sending the same updates to both release-controllers.  Usually, the architecture check prevents cross-contamination of the graphs.  Unfortunately, both origin and ocp are running against amd64.  This PR adds another check that ensures that the job's "Source" namespace matches the "releaseNamespace" of the release-controller.

/assign @smarterclayton

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale


[python] update pylint version

No changes needing a change description found.

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs

@iscai-msft before merge, could you open a pr to see if there is any impact for autorest.python?

waiting on completion of https://github.com/Azure/autorest.python/pull/2885 ci


Error when importing the plugin

An error occurred while trying to import and use your module:
import { generateAll } from './inc/rollup-images.js'
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at internalCompileFunction (node:internal/vm:74:18)
    at wrapSafe (node:internal/modules/cjs/loader:1141:20)
    at Module._compile (node:internal/modules/cjs/loader:1182:27)
    at Module._extensions..js (node:internal/modules/cjs/loader:1272:10)
    at Module.load (node:internal/modules/cjs/loader:1081:32)
    at Module._load (node:internal/modules/cjs/loader:922:12)
    at Module.require (node:internal/modules/cjs/loader:1105:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (G:\gulp_multitool\node_modules\posthtml-picture-srcset\index.js:1:18)
    at Module._compile (node:internal/modules/cjs/loader:1218:14)

I imported your module using import pictureSrcset from 'posthtml-picture-srcset'
I'm using your module in Gulp

.pipe(posthtml([
      easyBem(),
      pictureSrcset({
        imageOptions: {
          quality: 100,
          inputFormat: ['jpg', 'jpeg', 'png'],
          outputFormat: ['webp', 'avif'],
        }
      })
    ], {}))

In my package.json is a string "type": "module",

I think you just forgot to add a "type": "module", to your package.json. There was a time when I also made such a mistake.

Hey we just updated this, there was a single import inside that we've changed to require the function instead. Try the new version on NPM.

Great! But alas, that's not all.
I updated the module and got the following error:
Error: Cannot find module 'globby'
Require stack:
- G:\gulp_multitool\node_modules\posthtml-picture-srcset\lib\inc\rollup-images.js
- G:\gulp_multitool\node_modules\posthtml-picture-srcset\lib\pictureSrcset.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1039:15)
    at Module._load (node:internal/modules/cjs/loader:885:27)
    at Module.require (node:internal/modules/cjs/loader:1105:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (G:\gulp_multitool\node_modules\posthtml-picture-srcset\lib\inc\rollup-images.js:9:14)
    at Module._compile (node:internal/modules/cjs/loader:1218:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1272:10)
    at Module.load (node:internal/modules/cjs/loader:1081:32)
    at Module._load (node:internal/modules/cjs/loader:922:12)
    at Module.require (node:internal/modules/cjs/loader:1105:19) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    'G:\\gulp_multitool\\node_modules\\posthtml-picture-srcset\\lib\\inc\\rollup-images.js',
    'G:\\gulp_multitool\\node_modules\\posthtml-picture-srcset\\lib\\pictureSrcset.js'
  ]
}

I tried reinstalling node_modules, it didn't help.
I have not encountered this :(

I created a branch of my project so that you can install it for yourself and test it in my environment.
Don't forget to install the temporary branch.
https://github.com/Ulyanov-programmer/gulp_multitool/tree/temporary


Custom Error Type

Hi
It would be nice if this library would use a custom error type for all results, instead of mixing io::Error and Strings. This would play well together with the From<Error> trait.
http://blog.burntsushi.net/rust-error-handling/

On this issue, I would propose to use the std::io::Error directly, because most sources of errors come from reading/writing to the stream. The "unexpected response code" error could be described by an ErrorKind::InvalidData.
@mattnenterprise if you're interested and would accept a pull request, this is something that I'd like to implement. This is a very useful crate in my opinion, and I have a few proposals that I feel could improve it further, and that I would be happy to implement and contribute:

use a BufReader for parsing response codes. This simplifies read_response (thanks to the BufReader.read_line method), and should be more efficient than reading directly from the TcpStream.
support multiple line replies (a reply that starts with "ccc-" can contain multiple lines until the last line that begins with "ccc ", ccc being the same code as the one encountered on the first line).
similarly, use a BufWriter to write to the command stream.
I noticed you opened an issue to allow the FTPStream to be created with anything that implements Read and Write. There could be two constructors: one with a TcpStream that creates the FtpStream with reader and writer initialized to a BufReader/BufWriter from the stream, and another one that creates the FtpStream with a given reader and a given writer.
I suppose that FTPStream should be renamed to FtpStream from what I understand of Rust's conventions.
Perhaps use the same function names as Node's ftp module? "get", "put", "cdup", etc.
Have the "stor" function accept a closure with a writer as a parameter, to be able to upload data directly, rather than copying from a Read instance.

Let me know what you think about these ideas :smile:

I don't have anything to say, but I like @matt2xu s ideas :)

@matt2xu I like the ideas you have come up with. I think they would greatly improve this crate. I would gladly accept pull requests for these changes. I would one day like this crate to be the official FTP crate for rust, and I think these changes would make it closer to that goal.

@mattnenterprise great! I'll get started this week. This will actually be my first time contributing to another project on github, I look forward to it :-)

@matt2xu @dbrgn are you working on the error handling ? If not I'd give it a try.

@little-dude yes I have already implemented error handling, I was waiting for #17 to be merged first.
I think there are other things that you could work on that are potentially more interesting and quite useful too, in order of increasing complexity:

use status codes instead of hard-coded integers in read_response calls
rewrite current_dir to use iterators instead of inner functions
make pasv() use the host returned by the FTP server, rather than self.host (after which probably self.host wouldn't be needed anymore, and the constructor of FtpStream could just take a ToSocketAddrs directly)
change the API so that "stor" takes a closure for writing data. Not sure if "retr" should do the same or not?

Also, I proposed that we change some of the function names to match those in Node's ftp package, which are similar to the ftp command-line client: make_dir => mkdir, current_dir => pwd, change_dir => cwd, change_dir_to_parent => cdup, retr => get, stor => put, etc.
And of course, there's still #10 opened: implement rename, delete, append, system, etc.

Yeah, I think changing the method names to match the FTP commands makes sense. Then anybody that knows FTP will "feel at home".
I'm not working on anything else related to this library at the moment :) Maybe it's best if we create tracking issues for all of those points, if @mattnenterprise agrees that they should be implemented.

I agree that we should make the methods names match the ftp commands.
@matt2xu Could you make issues for the changes that you are proposing. So we can discuss those changes there, and it will be more organized.

@mattnenterprise :+1: I have created separate issues for these changes. This particular issue (error handling) is implemented in pull request #21. It's a big commit, but I feel it's worth it: the code is shorter and easier to read.

@mattnenterprise thanks for merging #21, I think we can close this issue as resolved!


Minimal image sizes aren't consistent

Minimal images are currently derived from the default ones, see: https://github.com/osbuild/cockpit-composer/blob/0201d2d80aff556f551619458d9497f6af21af59/components/Wizard/CreateImageUpload.js#L246
Default image sizes are basically hardcoded in cockpit-composer: https://github.com/osbuild/cockpit-composer/blob/0201d2d80aff556f551619458d9497f6af21af59/components/Wizard/CreateImageUpload.js#L295-L302
However, they definitely don't correspond to the default sizes in osbuild-composer:

Image type
RHEL 8.4
Fedora 33

ami
6
6

qcow2
10
2

openstack
4
2

vhd
4
2

vmdk
4
2

I wonder if should add a route to the Weldr API to return these default sizes and then load them in cockpit-composer. That shouldn't be super-hard. Thoughts? @jkozol @teg @bcl ?

Maybe make this part of the filesystem work? Or change this in the UI to say "minimal size", and then accept that composer will round up? Default it to 0 in the UI.

I wonder if should add a route to the Weldr API to return these default sizes

The data returned from /compose/types seems like the natural place to add them (using api/v1 of course). It looks like those numbers come from distro.ImageType.Size() so that should be fairly easy to add.


Finished styling the mentor dashboard sidebar and made some changes t…

…o the mentee dashboard header

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- src/components/MentorDashbord/common/MentorDashboardSidebarStyled.js  2

Complexity decreasing per file
==============================
+ src/components/common/UserCard/UserCard.js  -1

See the complete overview on Codacy


cannot delete the very beginning line (the first line) of the file

Hello,
I had a problem to delete the first line with command "dd". I cannot delete the first line with dd, and the CPU usage of VIM will become 100% after running dd in first line.
This is a python source code with .py as suffix.
1
2 def get_db():
3     from pymongo import MongoClient
4     client = MongoClient('localhost:27017')
5     db = client.myFirstMB
6     return db
7
8 def add_country(db):
9     db.countries.insert({"name" : "Canada"})
10
11 def get_country(db):
12     return db.countries.find_one()
13
14 if name == "main":
15     db = get_db()
16     add_country(db)
17     print get_country(db)
But it looks like this problem only happened in this special file. If I create a new one and it works.
I used Linux Mint as the following information:
Distributor ID:	LinuxMint
Description:	Linux Mint 18.2 Sonya
Release:	18.2
Codename:	sonya
Can you see if you can reproduce it and help fix it?
Thanks,
Shuai Hua

I found a work around to this issue:

find these two files: ".spf13-vim-3/.vimrc.bundles" and ".spf13-vim-3/.vimrc"
search keyword "python", and commented out all the lines with keyword "python"

It looks like there is a bug in python plugin such as "pythoncomplete", "python_match.vim", "python-mode" and "python.vim"
Thanks,
Shuai Hua


Binding two controls to one objects causes infinite loop

Describe the bug
When binding two controls two one object (e.g. to have one SelectedItem for two ListBoxes), updating the value causes an infinite loop.
To Reproduce
ViewModel:
public class MainWindowViewModel : ViewModelBase
{
	private MyModel _selectedModel;

	private IEnumerable<MyModel> Items { get; } = new[]
	{
		new MyModel(1, "First entry"),
		new MyModel(2, "Second entry"),
		new MyModel(3, "Third entry"),
		new MyModel(4, "Fourth entry"),
	};

	public MyModel SelectedModel
	{
		get => _selectedModel;
		set => this.RaiseAndSetIfChanged(ref _selectedModel, value);
	}
}

public record MyModel(int Id, string Name);

View:
<Window xmlns="https://github.com/avaloniaui"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:vm="using:AvaloniaRepro.ViewModels"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        mc:Ignorable="d" d:DesignWidth="800" d:DesignHeight="450"
        x:Class="AvaloniaRepro.Views.MainWindow"
        Icon="/Assets/avalonia-logo.ico"
        Title="AvaloniaRepro">

    <Design.DataContext>
        <vm:MainWindowViewModel/>
    </Design.DataContext>

    <StackPanel>
        <StackPanel Orientation="Horizontal">
            <StackPanel.Styles>
                <Style Selector="ListBox">
                    <Setter Property="ItemTemplate">
                        <DataTemplate DataType="{x:Type vm:MyModel}">
                            <TextBlock Text="{Binding Name}" />
                        </DataTemplate>
                    </Setter>
                </Style>
            </StackPanel.Styles>
            <ListBox Items="{Binding Items}"
                     SelectedItem="{Binding SelectedModel, Mode=OneWayToSource}"/>
            <Separator Width="1" Background="Black" />
            <ListBox Items="{Binding Items}"
                     SelectedItem="{Binding SelectedModel, Mode=OneWayToSource}"/>
        </StackPanel>
        <TextBlock Text="{Binding SelectedModel.Name, Mode=OneWay, FallbackValue='-'}" />
    </StackPanel>

</Window>

Click on any of the ListBoxItems to trigger the infinite loop.
Expected behavior
The selected value of the ListBox is displayed in the TextBlock.
Desktop (please complete the following information):

OS: macOS 12.2.1
Version 0.10.13

Additional context
May be related to #4438
When removing Mode=OneWayToSource from one of the controls, it works, however then the lists are synchronized.
When implementing the INotifyPropertyChanged pattern myself, the problem still occurs which suggests ReactiveUI is not the problem.

Hi, found the same problem using Mode=OneWayToSource, when i switch to Mode=TwoWay, the problem no longer happens.
Windows 10
Avalonia 0.10.13

Duplicate of https://github.com/AvaloniaUI/Avalonia/issues/4438


Add a license

Preferably Apache 2.0 :)

We recently went through an acquisition, so I need to get some guidance on this.  Will get back to you asap on adding a license.  I agree, I'd like to use Apache :)

If you need guidance on suitable licenses, go/opensource on the corp
network will be more than happy to assist you with licensing choices (even
if not related to opensource projects).
On Thu, Jul 9, 2015 at 8:47 AM, tzeskimo<EMAIL_ADDRESS>wrote:

We recently went through an acquisition, so I need to get some guidance on
this. Will get back to you asap on adding a license. I agree, I'd like to
use Apache :)
—
Reply to this email directly or view it on GitHub
https://github.com/pulseio/impala-base64/issues/1#issuecomment-120043551
.

@tzeskimo Thanks so much, and congratulations on the acquisition!


Microsoft authentication extension should not wrap console methods

Describe the bug
When developing or testing extensions, it's common for developers to log data from their extensions. However, because the Microsoft auth extension wraps console methods, the lines reported to the developer are incorrect.
To Reproduce
Steps to reproduce the behavior:

Create a hello world extension https://code.visualstudio.com/api/get-started/your-first-extension
In the launch.json, add the following lines (this will be unnecessary in the next release):"skipFiles": [
	"<node_internals>/**",
	"**/bootstrap-fork.js"
],

Add a console.log("hello") to the command in src/extension.ts
F5, and run the hello world command

Note: you may need to disable codespaces and the github auth plugin to see this, because they do the same thing https://github.com/github/codespaces/issues/5622 https://github.com/microsoft/vscode/issues/139895
Expected behavior
The line in the console points to the source line in the user's code where the log message appeared.
Actual behavior
It's the Microsoft auth extension where it wraps the global console methods. It should not wrap these methods.
Screenshots
See the source in the lower right corner points to a place where the extension wraps the global console methods.

I'm confused by this... AFAIK we don't do anything like that... maybe it's a bundling issue?
We use this:
https://github.com/microsoft/vscode/blob/main/extensions/microsoft-authentication/src/logger.ts
ok there's a single console.error here I should probably fix:
https://github.com/microsoft/vscode/blob/main/extensions/microsoft-authentication/src/authServer.ts#L66
but that's it.

My investigation for the GHPRI version of this issue pointed to https://github.com/microsoft/vscode-extension-telemetry/issues/88


Cloudbase-init cicustom broken with Proxmox 7.3 upgrade

I was using Cloudbase-init to provision VMs using Proxmox and this plugin using a guide here: https://yetiops.net/posts/proxmox-terraform-cloudinit-windows/
After upgrade to Proxmox 7.3 it no longer works. The cloud-init drive simply fails to show up.
Relevant  Terraform file:
resource "proxmox_vm_qemu" "prod-1" {
  name = "prod-1"
  target_node = "PVE-1"
  clone = "winserv2019-cloudinit"
  ipconfig0    = "ip=<IP_ADDRESS>/24,gw=<IP_ADDRESS>"
  cicustom = "user=local:snippets/prod-1.yml"
  cloudinit_cdrom_storage = "vm_data"
  os_type   = "cloud-init"
  memory       = 8192
  cores        = 8
  agent        = 1
  hastate      = "started"
  hagroup      = "ha_default"
  onboot       = true
  bootdisk = "scsi0"
  scsihw       = "virtio-scsi-pci"
  disk {
    size            = "100G"
    type            = "scsi"
    storage         = "vm_data"
    backup          = 1
  }
  network {
    model = "virtio"
    bridge = "vmbr110"
    firewall = "false"
  }
  lifecycle {
    ignore_changes = [
      disk
    ]
  }
}

Cicustom file /var/lib/vz/snippets/prod-1.yml
Content-Type: multipart/mixed; boundary="===============1598784645116016685=="
MIME-Version: 1.0

--===============1598784645116016685==
Content-Type: text/cloud-config; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="cloud-config"

set_timezone: America/Toronto
set_hostname: prod-1

--===============1598784645116016685==
Content-Type: text/x-cfninitdata; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="cfn-userdata"

#ps1

#enable unrestricted policy
Set-ExecutionPolicy Unrestricted

#disable administrator account
net user administrator /active:no

#wait for ping
do {
  $ping = test-connection -comp <IP_ADDRESS> -count 1 -Quiet
} until ($ping)

#setup jumpcloud
cd $env:temp | Invoke-Expression; Invoke-RestMethod -Method Get -URI https://raw.githubusercontent.com/TheJumpCloud/support/master/scripts/windows/InstallWindowsAgent.ps1 -OutFile InstallWindowsAgent.ps1 | Invoke-Expression; ./InstallWindowsAgent.ps1 -JumpCloudConnectKey "xxx"

#setup JC
#variables
$api = "xx"

#Install Nuget
Install-PackageProvider -Name NuGet -MinimumVersion <IP_ADDRESS> -Force

#import module
Import-Module -Name C:\windows\system32\WindowsPowerShell\v1.0\Modules\Microsoft.PowerShell.Management

#install JC
Install-Module -Name JumpCloud -Force
Connect-JCOnline $api -force

#sleep
Start-Sleep 120
#get system key
$test = select-string -Path $Env:Programfiles\JumpCloud\Plugins\Contrib\jcagent.conf -Pattern '"systemKey":"([0-9a-f]+)"' -AllMatches | % { $_.Matches } | % { $_.Value }
$test = $test.split(":")[1].Replace("`"","")

#connect to correct device group
Add-JCSystemGroupMember -GroupName 'NewProxmoxVMs' -SystemID $test

Pic of what is provisioned(notice lack of cloud-init drive):

The cloud-init drive is actually created, just not attached.
Debug log when I run terraform apply: https://gist.github.com/arhue/7e6440770dac6fa72c7849f73309a95c

This was some other issue, not to do with this provider. For some reason the RBD block device for cloud-init drive hadn't got deleted, triggering this issue. Deleting it fixed the issue.


com.baidu.rigel.biplatform.queryrouter.handle.manager.security.SecurityFilter  cannot be resolved

and also
com.baidu.rigel.biplatform.queryrouter.queryplugin.plugins.druid.sql.DruidSqlExpression cannot be resolved

yes,i don't know!


Added support for radiotherm humidity

Description:
Adds support for the radiotherm humidity attribute to be usable from HASS
Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.
[X] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Hi @zombielinux,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

This PR contains commits that are not yours, please clean up your branch
We are in the process of revamping the climate architecture (#23899). We will not be able to merge any related PR until that PR is done.

I'll close here since this PR seems to be the same as https://github.com/home-assistant/home-assistant/pull/25024.
For the future please close old PRs that have been replaced with new ones.


fix: not correctly resolving unnamed routes

Fixes a few edge cases where the breadcrumbs component couldn’t properly resolve unnamed routes. For some reason that’s unclear to me, routes that are definitely defined and loaded only resolve to a proper route location with a resolved path property (e.g. the actual mesh name instead of :mesh) if they have a name. For this reason, I’ve added a name property to all routes that didn’t have one, yet.
Furthermore, changes how breadcrumbs create route targets so that instead of { name: '...' }, they now use { path: '...' } as navigating to, for example, { name: 'zone-abstract-view' } will only navigate to that particular route without further resolving children routes for any routes with empty path. However, this does happen when navigating to paths which is what we want here. All in all, router.resolve has some surprising and confusing behavior that we address with these changes.
Signed-off-by: Philipp Rudloff<EMAIL_ADDRESS>

Issue: Turning this back into a draft as these changes introduce an issue with determining the active sidebar navigation item.


Fix orders of yaml of policy/pod-security-policy

The orders of kind were random, and that made the doc unreadable.
This fixes the orders in consistent way in pod-security-policy.md.
ref: https://github.com/kubernetes/website/issues/13862

/lgtm
/approve


Location information for typespec_member/enum_typespec

For code snippet below:
logic [62:36] wpri4;  // writes preserved reads ignored
xlen_e        sxl;    // xlen_e is an enum

here's the UHDM output -
|vpiTypespecMember:
\_typespec_member: (wpri4), id:313, line:57:23, endln:57:28
  |vpiName:wpri4
  |vpiTypespec:
  \_logic_typespec: , id:312, line:57:9, endln:57:14
    |vpiInstance:
    \_package: riscv (riscv::), id:0 dut.sv:1:1: , endln:671:11, parent:work@top, parID:13585
    |vpiRange:
    \_range: , id:309, line:57:16, endln:57:21, parent:riscv::status_rv_t, parID:306
      |vpiLeftRange:
      \_constant: , id:310, line:57:16, endln:57:18, parID:309
        |vpiDecompile:62
        |vpiSize:64
        |UINT:62
        |vpiConstType:9
      |vpiRightRange:
      \_constant: , id:311, line:57:19, endln:57:21, parID:309
        |vpiDecompile:36
        |vpiSize:64
        |UINT:36
        |vpiConstType:9

|vpiTypespecMember:
\_typespec_member: (sxl), id:314, line:58:23, endln:58:26
  |vpiName:sxl
  |vpiTypespec:
  \_enum_typespec: (riscv::xlen_e), id:279, line:42:5, endln:46:14
    |vpiName:riscv::xlen_e
    |vpiInstance:
    \_package: riscv (riscv::), id:0 dut.sv:1:1: , endln:671:11, parent:work@top, parID:13585
    |vpiBaseTypespec:
    \_logic_typespec: , id:278, line:42:18, endln:42:23
      |vpiInstance:
      \_package: riscv (riscv::), id:0 dut.sv:1:1: , endln:671:11, parent:work@top, parID:13585
      |vpiRange:
      \_range: , id:275, line:42:25, endln:42:28
        |vpiLeftRange:
        \_constant: , id:276, line:42:25, endln:42:26, parID:275
          |vpiDecompile:1
          |vpiSize:64
          |UINT:1
          |vpiConstType:9
        |vpiRightRange:
        \_constant: , id:277, line:42:27, endln:42:28, parID:275
          |vpiDecompile:0
          |vpiSize:64
          |UINT:0
          |vpiConstType:9
    |vpiEnumConst:
    \_enum_const: (XLEN_32), id:281, line:43:9, endln:43:25
      |vpiName:XLEN_32
      |UINT:1
      |vpiDecompile:2'b01
      |vpiSize:2
    |vpiEnumConst:
    \_enum_const: (XLEN_64), id:284, line:44:9, endln:44:25
      |vpiName:XLEN_64
      |UINT:2
      |vpiDecompile:2'b10
      |vpiSize:2
    |vpiEnumConst:
    \_enum_const: (XLEN_128), id:287, line:45:9, endln:45:25
      |vpiName:XLEN_128
      |UINT:3
      |vpiDecompile:2'b11
      |vpiSize:2

logic_typespec instances are unique for every use. However, enum_typespec uses are not. The former can hold location information relevant to the context of use but the latter cannot.
Would it be possible to convert the latter to use a ref_obj so that it can hold the context location information? i.e.
Instead of typespec_member -> enum_typespec, you would have typespec_member -> ref_obj -> enum_typespec.

Fixed by #2679.
typespec_member have new ref location fields:
#define  vpiRefFile          3018
#define  vpiRefLineNo        3019
#define  vpiRefColumnNo      3020
#define  vpiRefEndLineNo     3021
#define  vpiRefEndColumnNo   3022
The ref_typespec (ref_obj was not possible) solution would have broken too much existing code.

Noticed a similar issue with logic_typespec as well when declared as typedef.
Test: ArianeElab
https://github.com/chipsalliance/Surelog/blob/c85f65ef41ea7bff7a55ae8deddf63b27a95b7c6/tests/ArianeElab/dut.sv#L30
|vpiTypespecMember:
    \_typespec_member: (cause), line:937:30, endln:937:35, parent:ariane_pkg::exception_t
      |vpiName:cause
      |vpiTypespec:
      \_logic_typespec: (riscv::xlen_t), line:30:13, endln:30:18, parent:riscv::

Shouldn't this be a clone instead of using the existing instance? I need the location information of the typespec which would be wrong here.

@hs-apotell you made several fixes regarding location. This should be closed.


How to use [ ] in column names

Now Alasql supports column names enclosed with [ and ]. What if the column name has [ and ]?
Current RegExp for bracketed columns is:
    \[([^\]])*?\]

and it should be improved to accept something like escape character before ].

Now you can use ` to enclose such column names, like
    SELECT `my [column]` FROM ?

Nice ! thanks :+1:


Issue for check the empty files

Node-formidable seem to be has no method 'empty file checking'
Can Node-formidable checking for input files if it has a length or size of zero byte files?

Use allowEmptyFiles option


Minor corrections

The second cat command is missing the apiVersion line.
Should be:
cat <<EOF | kubectl apply -f -
apiVersion: networking.istio.io/v1alpha3

Also, in the current version of Istio, we need to run:
kubectl apply -f samples/bookinfo/networking/virtual-service-all-v1.yaml -n default
kubectl apply -f samples/bookinfo/networking/destination-rule-all-mtls.yaml -n default

instead of
kubectl apply -f samples/bookinfo/routing/route-rule-all-v1-mtls.yaml -n default

Thoughts on this?

Thanks for bringing this to my attention. I will make the changes in a day or two


Directory reorg

Reconfigured directory structure moving all widget components to sofi.ui and all main application components to sofi.app.
Added setup.py allowing python setup.py install and python setup.py test - Note: For some reason setuptools isn't pulling hypothesis like it does pytest when running with the test argument, so may have to manually pip install hypothesis before running tests.

Closes #29


import malaya issue

I think there's an issue when importing malaya. The issue arises in file pagerank.py line 3 from networkx import nx. I've tried to clone and change that line to import networkx as nx and manually install from the source. The issue has been resolve.

Cannot replicate, https://colab.research.google.com/drive/1HtLmcU3etpEKQFccBuGoHOYv5NSbgWtr?usp=sharing, should be not a problem

from networkx import nx
nx

<module 'networkx' from '/usr/local/lib/python3.7/dist-packages/networkx/init.py'>
import networkx as nx
nx

<module 'networkx' from '/usr/local/lib/python3.7/dist-packages/networkx/init.py'>


Dropdown api shows 'Selected undefined' message

Expected behavior
On successful selection of value taiko should display selected Audi where Audi is an option in drop down.
Actual behavior
taiko displays below message:
Selected undefined
Steps to reproduce

openBrowser()
goto("file:///Users/data/HTMLElements.html")
dropDown("Cars").select("Audi")

Kindly find below html snippet:
<label for="select">Cars</label>
    <select id="select">
        <option value="volvo">Volvo</option>
        <option value="saab">Saab</option>
        <option value="mercedes">Mercedes</option>
        <option value="audi">Audi</option>
    </select>

Versions
Version: 1.0.1 (Chromium: 76.0.3803.0) 6b81b87720543bc9a2e12cb9649784a7be82f5c6

This issue has been verified and found fixed.
Tested version:
Version: 1.0.1 (Chromium: 76.0.3803.0) 5ada00f72c218606b9681f18966987ac606bcaba
Kindly find the below output:
Version:  (Chromium:)
Type .api for help and .exit to quit

openBrowser()
✔ Browser opened
goto("file:///Users/data/HTMLElements.html")
✔ Navigated to URL file:///Users/debas/dadebash/work/taikoProject/25junTaikoRepo/taiko/test/functional-tests/specs/data/HTMLElements.html
dropDown("Cars").select("Audi")
✔ Selected Audi


New logo

I've changed the logo to remove dregs.
ill also go through and change some of the primary front facing things to remove the reference.
i dont see a big issue with leaving it as a short hand for dataregstiry deeper in the docs and the name of the CLI command?

I think it's fine to keep DREGS for the CLI command, but I would rather not use it as shorthand for the actual facility anywhere.

@JoanneBogart i think i've removed all mention of dregs, other than it being the CLI command.
anywhere i've missed?


DI For Middleware

ng-redux for Angular 1 supports DI for middleware, would like to get the same functionality here.
Prior art/other examples:
ngrs/Store and how they handle middleware
angular2-redux - although don't think this lets middleware use DI
ng-redux
Thinking so we can still keep with the provide(store), maybe something like:
import { provide, createStoreWith } from 'ng2-redux'

let store = createStoreWith(reducer,middleware,storeEnhancer, initialState)
bootstrap(App,provide(store));

or
import thunk from 'redux-thunk'
import someProviderMW from 'some-provider-mw'

let store = createStoreWith(reducer,[thunk,someProviderMW])
bootstrap(App,provide(store));

Then - if the middleware is just a function - push it into the middleware array, if the middleware is an instance of a provider - use the injector to create an instance of it, then push it onto the middleware collection.
use case: making a middleware that needs access to Angular's HTTP service,

So ngrx and redux middlewares are actually quite differerent.  Ngrx lets you add 'pre' and 'post' middlewares, and they appear to be similar to RxJS operators: (observable: Observable) => Observable
Redux middlewares are in practice essentially curried versions of (store, next, action) => action
Nonetheless, I like ngrx's idea of providing a 'createMiddleware' convenience function to handle the DI if needed.  Taking a look at what that would entail. The signature would be something like:
type <S>createMiddleware(middleware: Middleware<S>, providers: Provider[]): Middleware<S>

In that case you could do:
import thunk from 'redux-thunk'
import someMwWithDiDependencies from 'some-mw'

let store = createStoreWith(reducer,[thunk,createMiddleware(someMwWithDiDependencies)])
bootstrap(App,provide(store));

Looking into it more I don't think we can do this without wrapping applymiddleware.
The reason is that we're currently applying middleware functions at boot time, before the injector has been created.  However in order to go from a middleware factory function that takes injectable arguments, we need to have an injector.  So it's a chicken and egg problem.
Really the sequence needs to be:

register providers for middlewares than need DI with Angular
register these mw factories also with ng2-redux
bootstrap the app
after bootstrap, have ng-redux run through it's list of mwFactories and convert them in to middlewares, and apply them to the store.

The way redux works, we'll start to go down a rabbit hole of delaying store configuration until after bootstrap as well, which is a pretty large change.
I think we need to delay this one until post release.

NgRedux gets away with it because they wrap pretty much everything; applyMiddleware is no longer part of store configuration - you can call usePreMiddleware and usePostMiddleware after the fact.
Since they also hold references to their middle wares they can fiddle with the injection later as well.
Since we actually don't wrap the store config and middleware application at all, but use the vanilla redux stuff our flexibility is limited.  If we could somehow separate provide(store) and middleware application we'd be in business.

We could do something like:
import { Middleware } from 'redux';
import { RandomNumberService } from '../services/random-number';
import { Injector } from 'angular2/core';

const inj = Injector.resolveAndCreate([RandomNumberService]);

export const diMiddleware = store => next => action => {
  const rns = inj.get(RandomNumberService);

  console.log('Pre-number:', rns.pick());
  const result = next(action);
  console.log('Post-number:', rns.pick());
  return result;
};

But it's not clear how this fits into the injector tree.  I think it creates a new one. This could lead to things that are supposed to be singletons not being any more.
Also I'm not really sure how you'd test this.

Another alternative would be to make ngRedux @Injectable and only talk to redux through it.
But in this case, we'd bootstrap like this:
bootstrap(App, [ NgRedux, ... ]

And then delay all the configure store stuff to the App component:
@Component({
  ...
})
class App {
  constructor(private ngRedux: NgRedux) {
    ngRedux.createStore(reducers, middlewares, enancers, initialState);
  }
}

In this case createStore would do the actual redux stuff on our behalf, and would at this point have the ability to intercept middlewares whose dependencies need resolving and resolve them first before passing them to redux.
However this would likely not be backward compatible (unless we can keep the only init strategy around too, maybe with a deprecation warning).

This way, the store isn't directly injectable - you'd have to inject ngRedux and call .getStore() or similar.

It would also complicate an @Select implementation somewhat, although we already have DI issues with that.

Clearly this needs some discussion.  In the interim, I've provided a README example for how this can be done with the current beta.

https://github.com/angular-redux/ng2-redux/pull/49

Another option is to get rid of provide(store) and ngRedux's constructor param and instead provide an ngRedux.setStore() method.
This way we can construct the store and provide the middlewares after the DI has bootstrapped, most likely in the app's root component:
@Component({
  // ...
})
class App {
  constructor(
    private ngRedux: NgRedux<IAppState>,

    ngRedux.setStore(configureStore({}));
  );
}

Pros:

Nothing else changes in the ngRedux API.
Simple to implement.
At this point we can use injection wherever we feel like it: middlewares, enhancers, etc.
Everything inside ngRedux can use angular services if needed - this opens the way for ngZone devtool fixes, etc.

Cons:

Breaks backwards compatibility: we'd need to do a major version bump.
Need to think through what the effect is on a potential @Select implementation.
Nothing preventing someone from calling setStore twice - we'll need to guard against that.

@Select would likely need to create an empty observable and register it with ngRedux global state somehow.  When setStore() gets called, we'd need to hook chain the empty observable up to an actual observable.
Also not clear when these global refs would get cleaned up?


LightningTemplateModel is broken

🐛 Bug
LightningTemplateModel has no implemented validation_step_end method. See here
To Reproduce
Just try to run test evaluation on MNIST example
Expected behavoir
It should not crash

@festeh mind draft a PR to fix it? :rabbit:

@Borda yes, will make it soon


How is the ethscription number determined?

I have looked around the docs at https://docs.ethscriptions.com/overview/introducing-ethscriptions but didn't find the ethscription number and valid formats rule.
How is the ethscription number determined? what data formats are valid ethscriptions?

Ethscription numbers aren't an official part of the protocol—ie nothing about the protocol depends on their existing. They are more of a "for fun" thing that indexers can choose to implement or not (and users should take with a grain of salt and not use as identifiers).
We define the ethscription number for ethscription X as "the number of valid ethscriptions before X" where before means "in an earlier block or in the same block with a lower transaction index."
However, this definition itself is "loose" in that we only calculate this once every 20 mins or so, meaning if you query us you will not always get the same thing, which is obviously not acceptable for any real part of the protocol.
The reason is that these numbers are quite difficult to calculate as you can't just keep a running total because of re-orgs.

Can we establish a rule for Ethscription numbers similar to Ordinals?
Using the transaction hash as the Ethscription ID is inefficient.
If the ID rules of the Ethscription are unclear, it hinders the development of related applications and the utilization of the combination between projects.

The issue is that the Ethscription Number can change during a reorg and
Ethscription ids must never change.
But people can use the numbers if they want! The “show ethscription” API
endpoint allows you to use them instead of the transaction hash. But if you
want to uniquely identify an ethscription using an identifier that will not
change, Ethscription Numbers are a poor choice because of the way Ethereum
works.
Does this make sense? I admit I don’t know how Ordinals solves this problem
as there are reorgs on Bitcoin as well.
On Sun, Dec 17, 2023 at 3:45 AM xfu @.***> wrote:

Can we establish a rule for Ethscription numbers similar to Ordinals?
Using the transaction hash as the Ethscription ID is inefficient.
If the ID rules of the Ethscription are unclear, it hinders the
development of related applications and the utilization of the combination
between projects.
—
Reply to this email directly, view it on GitHub
https://github.com/ethscriptions-protocol/ESIP-Discussion/issues/12#issuecomment-1859075690,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAOONXUS2257HN7AJ7N64TYJ2WMLAVCNFSM6AAAAABAOH5GBCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJZGA3TKNRZGA
.
You are receiving this because you commented.Message ID:
@.***>

I don't know the Ethscription number context. Why will the number change after reindexing the block?
There are two options for the Ethscription number:

Ethscription number = block number + tx index * 100000
Starting from 0, increment each Ethscription by one.
For the option 2:
Define what constitutes a valid Ethscription. A valid Ethscription should adhere to the DataURI scheme. For example: data:image/png;base64, or data:application/json,. Thus, a valid Ethscription is data:*;*.
Indexing starts from block 0 and transaction (tx), and the Ethscription number increments.
I think the step 2 should yield a deterministic number.

The issue is that chain reorgs can affect tx order and therefore ethscription number in both of your proposals. Example:

You observe me ethscribe a beautiful image and note the ethscription number is 100.
The next day you see someone selling ethscription number 100! You buy it.
But no! There was a reorg and now number 100 points to a different ethscription than it did when it was first ethscribed. You just bought the wrong asset!

This is the nature of Ethereum and so ethscription numbers can never be relied on as ids, but rather used "for entertainment only."
Does this make sense?

The issue is that chain reorgs can affect tx order and therefore ethscription number in both of your proposals. Example:

You observe me ethscribe a beautiful image and note the ethscription number is 100.
The next day you see someone selling ethscription number 100! You buy it.
But no! There was a reorg and now number 100 points to a different ethscription than it did when it was first ethscribed. You just bought the wrong asset!

This is the nature of Ethereum and so ethscription numbers can never be relied on as ids, but rather used "for entertainment only."
Does this make sense?

Thank you for your answer.
Now i understand what's Ethereum's reorg is.
https://beaconscan.com/epochs

The transactions will be finally confirmed after the epoch vote, and the final confirmation time will be delayed by approximately 20 minutes compared to the execution nodes. Therefore, the confirmation of the Etherscription number should be determined after the final confirmation of the epoch. After epoch confirmation, there will be no more reorg situations for transactions.

The issue is that the Ethscription Number can change during a reorg and Ethscription ids must never change. But people can use the numbers if they want! The “show ethscription” API endpoint allows you to use them instead of the transaction hash. But if you want to uniquely identify an ethscription using an identifier that will not change, Ethscription Numbers are a poor choice because of the way Ethereum works. Does this make sense? I admit I don’t know how Ordinals solves this problem as there are reorgs on Bitcoin as well.
…
On Sun, Dec 17, 2023 at 3:45 AM xfu @.> wrote: Can we establish a rule for Ethscription numbers similar to Ordinals? Using the transaction hash as the Ethscription ID is inefficient. If the ID rules of the Ethscription are unclear, it hinders the development of related applications and the utilization of the combination between projects. — Reply to this email directly, view it on GitHub <#12 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAOONXUS2257HN7AJ7N64TYJ2WMLAVCNFSM6AAAAABAOH5GBCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJZGA3TKNRZGA . You are receiving this because you commented.Message ID: @.>

https://github.com/ordinals/ord/blob/d791800fa438469ac8102f8f972e2a1e99b16bd4/src/index/reorg.rs
This is related to the reorganization process of Ordinals.

The issue is that the Ethscription Number can change during a reorg and Ethscription ids must never change. But people can use the numbers if they want! The “show ethscription” API endpoint allows you to use them instead of the transaction hash. But if you want to uniquely identify an ethscription using an identifier that will not change, Ethscription Numbers are a poor choice because of the way Ethereum works. Does this make sense? I admit I don’t know how Ordinals solves this problem as there are reorgs on Bitcoin as well.
…
On Sun, Dec 17, 2023 at 3:45 AM xfu @.> wrote: Can we establish a rule for Ethscription numbers similar to Ordinals? Using the transaction hash as the Ethscription ID is inefficient. If the ID rules of the Ethscription are unclear, it hinders the development of related applications and the utilization of the combination between projects. — Reply to this email directly, view it on GitHub <#12 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAOONXUS2257HN7AJ7N64TYJ2WMLAVCNFSM6AAAAABAOH5GBCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJZGA3TKNRZGA . You are receiving this because you commented.Message ID<EMAIL_ADDRESS>
https://github.com/ordinals/ord/blob/d791800fa438469ac8102f8f972e2a1e99b16bd4/src/index/reorg.rs
This is related to the reorganization process of Ordinals.

https://github.com/ordinals/ord/pull/2320

If a reorg is detected it rolls back the database 6 blocks and reindexes from there.

@RogerPodacter indexers can choose to implement or not (and users should take with a grain of salt and not use as identifiers).

OOH! That's a relief. :laughing: Was just thinking about that recently.

@0xxfu Using the transaction hash as the Ethscription ID is inefficient.

Inefficient in what manner? Transaction hashes are universally accepted everywhere and the whole ecosystem can work with them already.

@0xxfu Can we establish a rule for Ethscription numbers similar to Ordinals?

I've been thinking recently (and i even built it into the indexer i'm open sourcing in a bit) to have unique IDs - secure, stable, shorter, and collision resistant - cuid v2.
This can work as "sat names" in fact and who knows what more interesting things. The default length is 24 alphanumeric characters like ek0yimz8ihx6gyljz151ihyl. And the explorer i'm building will also support them. It's not a lot better than transaction hashes but at least a bit.
What you think @RogerPodacter Middle To propose it as ESIP? We can be strict and define them to be with length of 15 or 20 (bigger is better but harder for "end users" ;d).

Interesting, @tunnckoCore!
I think if we want shorter ids we should tie them to the transaction hash in some way for backwards compatibility. Eg we could encode the 32 bytes as base64 and drop the 0x which would reduce the length from 66 to 43 with no loss in fidelity.
If you wanted to go further you could cut that in half for 128 bits of id space versus 256, which is also more than enough and at least this would give you the ability to convert one-way.
But I don't think the juice is worth the squeeze to force everyone to support a new id scheme now. For that I'd think we need to add a major new feature like multiple ethscriptions per tx which I know I've said I never ever ever ever want to do but who knows never say never.

@RogerPodacter we need to add a major new feature like multiple ethscriptions per tx which I know I've said I never ever ever ever want to do but who knows never say never.

HA HA HA HA!! Yeah. People are always coming to what WGW is asking or saying :D
Interesting turn of events, I was in support before, but not that much now and kinda relieved cuz recently just thought how harder it gonna be for indexers. Well, not that much tho.. It's only a matter of supporting it through events, which is easy to track so.. :man_shrugging:


stats: Improve error message for bad snapshot ID (fixes #1933)

What is the purpose of this change? What does it change?
Improves the error handling when restic stats is given a bad snapshot ID.
Was the change discussed in an issue or in the forum before?
#1933
Checklist

[x] I have read the Contribution Guidelines
[ ] I have added tests for all changes in this PR
[ ] I have added documentation for the changes (in the manual)
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[x] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

Not sure if you really need the changelog file for a minor text change -- let me know if that's the case.

Codecov Report

Merging #1936 into master will decrease coverage by 4.57%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1936      +/-   ##
==========================================
- Coverage   50.56%   45.98%   -4.58%
==========================================
  Files         170      170
  Lines       13213    13213
==========================================
- Hits         6681     6076     -605
- Misses       5529     6189     +660
+ Partials     1003      948      -55

Impacted Files
Coverage Δ

internal/restic/backend_find.go
47.22% <ø> (ø)
:arrow_up:

cmd/restic/cmd_stats.go
3.96% <0%> (ø)
:arrow_up:

internal/backend/b2/b2.go
0% <0%> (-78.54%)
:arrow_down:

internal/backend/swift/swift.go
0% <0%> (-78.45%)
:arrow_down:

internal/backend/gs/gs.go
0% <0%> (-73.61%)
:arrow_down:

internal/backend/azure/azure.go
0% <0%> (-69.46%)
:arrow_down:

internal/backend/swift/config.go
36.95% <0%> (-54.35%)
:arrow_down:

internal/archiver/file_saver.go
81.57% <0%> (-4.39%)
:arrow_down:

internal/archiver/blob_saver.go
92.59% <0%> (-2.47%)
:arrow_down:

internal/backend/s3/s3.go
60.15% <0%> (-2.26%)
:arrow_down:

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 beb208e...6f5b0f3. Read the comment docs.

Hrmmmm... it's 2018 and I can't change the branch that a PR originates from on GitHub of all places? Sad day. Closing in favor of #1942.


[WW-4972] Switches incompatible_improvements to VERSION_2_3_28

Implements WW-4972

Coverage remained the same at 46.93% when pulling 796c15ee673247fe6c0c23da0b4e29a6ec4bb29d on lukaszlenart:WW-4972 into 99b97f1136a3ac60773153b48834a455799c93c9 on apache:master.

Travis jdk7 fails on updateimpact submit - I have no vision on updateimpact

@lukaszlenart What about ?html in other files?

@aleksandr-m to be honest I'm not sure if this is a proper solution at all... for other side, those are only JS related attributes.

@lukaszlenart Me neither. Assuming everything works as it is now, maybe we can postpone removing of ?html until it is removed from freemarker?

It's rather an issue with changed behaviour of ?html - when I leave it, all single quotes will be escaped with a proper entity.
And now I'm reading about ?html and it was marked as deprecated, I think I will extend this PR and will throw out all those in return using auto-escaping

@aleksandr-m I have dropped all the ?html and switched into auto escaping mode by default. I was wonder if instead of hardcoding auto escape into FreemarkerManager it would be better to rename all .ftl files into .ftlh - thus enables HTML auto escaping by a file type.

@lukaszlenart I think it is ok. Renaming templates means that users also must rename overridden templates in their apps. It is not a big deal, but still one more thing to do upon upgrading. Besides each template can change these settings with <#ftl output_format=...> and <#ftl auto_esc=...>.

I just figured out that I should also review all the JS related attributes to be sure to use
<#outputformat 'JavaScript'>${parameters....}</#outputformat>

instead of enabling JS escaping for the whole template.

@apache/struts-committers this is ready for another round


web interface issues

When clicking the gears icon in the 2.7 web interface, the design looks broken, at least in Firefox 40.
See attached screenshot. The problem exists when clicking the gears icon in the Collections tab and all the other tabs.

Same in Chrome 47. Looks to me like there's a CSS style missing to disable the default list bulletins.


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20230206164004959 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20230206164004959 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
28663

Relevant Lines:
63351

💛  - Coveralls


rbd: add EncryptionLoad api to support the rbd_encryption_load librbd…

rbd: add EncryptionLoad api to support the rbd_encryption_load librbd api.
EncryptionLoad loads the encryption format information for an image and enables encrypted IO.  This is a new feature in Ceph Pacific.
Signed-off-by: Effi Ofer<EMAIL_ADDRESS>
Checklist

[x] Added tests for features and functional changes
[x] Public functions and types are documented
[x] Standard formatting is applied to Go code

One more small nitpick about the tests: you write the data with encryption enabled, and read it back from the same open image. I think it would be better for verification to close the image after writing and then reopen it with encryption enabled. That way we're unlikely to be hitting any caches, etc.
If you don't think this is a good idea, please let me know. Otherwise things are looking good!

Done

@phlogistonjohn thanks for the review.  Any idea why the test-suite is failing?

@phlogistonjohn thanks for the review. Any idea why the test-suite is failing?

Unfortunately, no. It seems like it could be a time out. It certainly was the first time, but the most recent time as of this writing is more mysterious. I've started the CI again. If if fails again in a similar way I'll investigate deeper, but I'm hoping we're just getting unlucky with test flakes.

Good news: it looks like we did just get unlucky with the previous two runs, the CI has passed now.

Good news: it looks like we did just get unlucky with the previous two runs, the CI has passed now.

That's awesome and thank you @phlogistonjohn.  What do we need to do next to get this merged?

Good news: it looks like we did just get unlucky with the previous two runs, the CI has passed now.

That's awesome and thank you @phlogistonjohn. What do we need to do next to get this merged?

A second approving review. Please be patient, as most of our maintainers do go-ceph work as part of their jobs and I know some people are unavailable today.

Good news: it looks like we did just get unlucky with the previous two runs, the CI has passed now.

That's awesome and thank you @phlogistonjohn. What do we need to do next to get this merged?

A second approving review. Please be patient, as most of our maintainers do go-ceph work as part of their jobs and I know some people are unavailable today.

No rush.  And thanks for responding so quickly.

@mergifyio rebase


Add ability to set sourceLocale through the config file

Add the ability to set sourceLocale through the config file

Please add some release notes under the next release, v1.14.0. Otherwise, this is approved.

The README.md file is getting longer. so I created the new ReleaseNotes.md file.


[Bug] [ingestion-server] Tldextract caching error

Description

Error message from production:

2021-07-09 16:31:52,160 WARNING cache.py:166 - unable to cache publicsuffix.org-tlds.{'urls': ('https://publicsuffix.org/list/public_suffix_list.dat', 'https://raw.githubusercontent.com/publicsuffix/list/master/public_suffix_list.dat'), 'fallback_to_snapshot': True} in /home/supervisord/.cache/python-tldextract/3.9.6.final__local__ecb11d__tldextract-3.1.0/publicsuffix.org-tlds/de84b5ca2167d4c83e38fb162f2e8738.tldextract.json. This could refresh the Public Suffix List over HTTP every app startup. Construct your `TLDExtract` with a writable `cache_dir` or set `cache_dir=False` to silence this warning. [Errno 13] Permission denied: '/home/supervisord'
2021-07-09 16:31:52,623 INFO cleanup.py:162 - https://www.flickr.com/photos/35762655@N04:200
2021-07-09 16:31:52,624 INFO cleanup.py:80 - Tested domain www.flickr.com

As this error report suggests, our docker user isn't able to write to the default tld suffix list. This causes the cleanup job to take significantly longer, as each round it needs to download the public suffix list.
Reproduction

Should be reproducible by attempting to run the curl -XPOST localhost:8001/task -H "Content-Type: application/json" -d '{"model": "image", "action": "INGEST_UPSTREAM"}' task, at which point the above error would be visible during the cleaning process.
Resolution

[ ] 🙋 I would be interested in resolving this bug.

This cURL request is run as a part of the load_sample_data.sh script. But I have not seen this message appear in the logs for that script.

This seems related to the production infra, and not the api itself. Closing this and will create a new ticket there.