Can you pull 1 minute data with Data on demand?

is it possible to pull aggregated minute data with the data on demand product ? or only ticks data? cause I noticed there is an End of day endpoint, but not sure if it's only for the whole day or you can give it intraday time ranges as well ?
Thanks!

SDK on this repo provides ticks data with streaming API based on apache kafka.
It is possible to pull aggregated minute data with the data on demand product using the summarized trades endpoint. You can set the trade_period as 1 and the trade_precision as "MINUTE".  This will return aggregated data for every minute within your time range.
The documentation for the same is : https://dataondemand.nasdaq.com/docs/index.html#summarized-trades-data .
Please let us know if you need more information.

Yes! that is what I thought, it's not that clear though haha. What is the first sentence you wrote in the message here about the SDK, did not understand it ?
also, yes I have a few more questions on this:

In the docs it says 'First trade' and 'Last trade'. Does that mean OHLCV ? bcs it does not state it like that. I am talking about a way to pull aggregated data with all the eligibility rules etc'. In the end of day section it does refer to it as OHLCV but in the summarized it does not for some reason, just want to make sure it will return valid 1 minute bars.

another thing is here in this doc : http://www.nasdaqtrader.com/content/productsservices/dataproducts/dod/data-on-demand_faq.pdf . it says that one day of trades data is 0.201 GB for all US equities. We have some data and that is clearly in correct as it's somewhere between 4-5GB (only trades). So wanted to know how do those numbers were determined ? 1 year of trades is 52 GB ? does not seem right to me, can you double check?

Is there a way to get a trail to test the API endpoints for a day or 2?

Is the historical data unadjusted or adjusted data? for splits etc.

Does the data contain delisted and name changes tickers? so if a ticker was delisted in 2018 for example will you have it until that point? same as for symbol changes ?

Do we have to set up a UTP agreement for redistribution ?

Thanks, hoping to hear from you as soon as you can.

Also, what is the best way to subscribe to the product ?

We provide streaming service for data as well as REST api service.
This repo has an SDK for streaming service. Where you can subscribe to the different streams.
We also have REST api service for requesting trades, quotes and analytics. https://dataondemand.nasdaq.com/docs/#introduction.

@ruchirvaninasdaq can you answer my other questions pls ?

@noam2 : You can send an email with all question to<EMAIL_ADDRESS>You will be able to get all answers and subscribe to the product also.

Can you address some of the technical questions here about the data? @ruchirvaninasdaq


Correct spelling of Repository in fail.html

This checklist is used to make sure that common guidelines for a pull request are followed.

[ N/A ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ X ] I adhere to the Command Guidelines.

@williexu rebased off dev


A way to change the AdGuard Statistics Port

Description
Hey,
I have adguard running over https and on a completly different port and I think that causes the Statistics to not show at all.
I would wish for a way to change that
Please tick the boxes

[X] You've read the docs
[X] You've checked for duplicate issues

Sorry, I don't quite understand your problem here?
As long as you point homarr to the IP:Port of adguard, it should be ok.
Note that you will most likely need to use http if you don't have valid certificates for https.

I dont see where I can set the IP:Port or anything for the Widget

You have to set the address in the app. Widgets just make use of your apps.


Allow easily copy installation steps from docs

Allow easily copy installation steps from docs
Copy scripts from README.md file

@chrfalch Hey. Done


Buffer cache miss if the last block is less than 4K

Hey,
I noticed that for a simple file weighing less than 4K (default block size), a call to Gdrive API was always done even if the file was cached on disk (I set cache age to 24hours).
Found the culprit here : https://github.com/dweidenfeld/plexdrive/blob/master/buffer.go#L124
readAt returns io.EOF if EOF is encountered while reading buf.size (4096).
The same happens for a file with 5000 bytes or any chunk not weighing a multiple of 4096.
I changed the line like this :
if n, err := f.ReadAt(buf, fOffset); n > 0 {
Seems to do the job, no API call are made.
I could do a PR if u want.
Thanks

Just did it and referenced your username.

Fixed for next version.


[release-1.8] pkg/queue: optimize the delay queue and resolve some TODOs

This is an automated cherry-pick of #29226

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio_release-1.8
92da600689a3cd263aebfad29bb52499b0834a6c
link
/test integ-security-multicluster-tests_istio_release-1.8

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Fix : comment

Hello @j4rj4r! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file BotTwitter/retweet_giveaway.py:

Line 199:80: E501 line too long (92 > 79 characters)
Line 205:80: E501 line too long (92 > 79 characters)
Line 212:80: E501 line too long (92 > 79 characters)


vext.gi doesn't work with Python 3.10

In a virtual environment created with Python 3.10 (python3.10 -m venv my-venv) and initialized with up-to-date versions of pip, setuptools, and wheel (python3.10 -m pip install --upgrade pip setuptools wheel - wheel necessary as per #84), plus vext and vext.gi of course, running vext --check gi.vext fails even though vext is enabled according to vext --status:
(my-venv) $ vext --status
Running in virtualenv [enabled]
(my-venv) $ vext --check gi.vext
import gi.repository: [failed]

This was not the case with Python 3.8 (and I didn't try 3.9).
Full traceback below:
(my-venv) $ python3
Python 3.10.8 (main, Oct 12 2022, 19:14:26) [GCC 9.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import gi.repository
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/me/my-venv/lib/python3.10/site-packages/vext/gatekeeper/__init__.py", line 209, in load_module
    module = imp.load_module(name, *self.module_info)
  File "/usr/lib/python3.10/imp.py", line 245, in load_module
    return load_package(name, filename)
  File "/usr/lib/python3.10/imp.py", line 217, in load_package
    return _load(spec)
  File "/usr/lib/python3/dist-packages/gi/__init__.py", line 42, in <module>
    from . import _gi
ImportError: cannot import name '_gi' from partially initialized module 'gi' (most likely due to a circular import) (/usr/lib/python3/dist-packages/gi/__init__.py)

Sadly I am unable to fix this issue.

Interesting, and can you confirm pygobject works in the system python3.10

Good call: it does not, and the reason might have something to do with the fact that my system Python 3.10 is provided by the deadsnakes PPA on an Ubuntu 20.04 LTS system where the default system Python is 3.8.
$ python3
Python 3.8.10 (default, Jun 22 2022, 20:18:18)
[...]
>>> import gi.repository

$ python3.10
Python 3.10.8 (main, Oct 12 2022, 19:14:26)
[...]
>>> import gi.repository
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3/dist-packages/gi/__init__.py", line 42, in <module>
    from . import _gi
ImportError: cannot import name '_gi' from partially initialized module 'gi' (most likely due to a circular import) (/usr/lib/python3/dist-packages/gi/__init__.py)

No idea how to fix that, but in any case, not caused by vext, so we can close this issue.

No idea how to fix that

Old StackOverflow posts hint at creating 3.10 symlinks to the 3.8 shared libraries of _gi and _gi_cairo in the system's Python installation:
cd /usr/lib/python3/dist-packages/gi
sudo ln -s _gi_cairo.cpython-38-x86_64-linux-gnu.so _gi_cairo.cpython-310-x86_64-linux-gnu.so
sudo ln -s _gi.cpython-38-x86_64-linux-gnu.so _gi.cpython-310-x86_64-linux-gnu.so

But (unsurprisingly) this doesn't work, probably because the fake 3.10 libraries created that way are not the expected actual 3.10 ones:
$ python3.10
Python 3.10.8 (main, Oct 12 2022, 19:14:26)
[...]
>>> import gi.repository
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3/dist-packages/gi/__init__.py", line 42, in <module>
    from . import _gi
ImportError: /usr/lib/python3/dist-packages/gi/_gi.cpython-310-x86_64-linux-gnu.so: undefined symbol: _PyTraceMalloc_NewReference

Yeah if you are linking from a library in a virtual env (with symlinks or vext) to a system Python, they have to be exactly the same build.
A virtualenv with system-site-packages might be something you can try falling back to.

Makes sense. A virtual environment with system-site-packages would work indeed, although I was trying to avoid that.
Closing this issue now, as it is not an issue with vext after all.


Support login for multiple users

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes https://github.com/opensrp/fhircore/issues/2330
Engineer Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the FHIRCore app to verify my change fixes the issue and/or does not break the app
[ ] I have checked that this PR does NOT introduce breaking changes that require an update to Content and/or Configs? If it does add a sample here or a link to exactly what changes need to be made to the content.

Code Reviewer Checklist

[ ] I have verified Unit tests have been written for any new feature(s) and edge cases
[ ] I have verified any strings visible on UI components are in the strings.xml file
[ ] I have verifed the CHANGELOG.md file has any notable changes to the codebase
[ ] I have verified the solution has been implemented in a configurable and generic way for reuseable components
[ ] I have built and run the FHIRCore app to verify the change fixes the issue and/or does not break the app

@LZRS is this ready for review?

@LZRS is this ready for review?

It can be reviewed but there are still some code changes pending, as we look to consult further (with tpms) on how/what user roles to check for and also update tests

Currently, only supports multiple user logins when online. Will be adding support for offline as well in subsequent sessions, as we continue to discuss on how to effectively handle the pin login usecase

I think we can skip this UI/UX given that after one logs in using username and password we already know who the user is. The workflow for switching users should be like Logout -> Login in (for other user) -> Enter pin for username . We can infer the username from the previous (login) step.

I think we can skip this UI/UX given that after one logs in using username and password we already know who the user is. The workflow for switching users should be like Logout -> Login in (for other user) -> Enter pin for username . We can infer the username from the previous (login) step.

@ndegwamartin This would mean that users wouldn't be able to change their pin once setup, right? The user dropdown was intended to enable switching accounts using the pin login page without having to go to the username login page(and having to re-enter their password)

@LZRS I get it now, if that's the use case then yeah lets implement it like it is in the new design cc @Rkareko

@LZRS this ready for review?

@HenryRae cc @rowo Concerning login for multiple users there is need for user selection when performing login using a PIN. We are seeking your guidance on the UI for the user selection widget in the Pin Login screen. As shown above it has been implemented as a drop down menu. Is this sufficient? Could you provide mock ups for how best to implement this.

@Rkareko my concern with this approach is:
When user logs out, ideally they should only be able to access the app when they log back in the app using username & password. Not with just PIN
PIN login was for quick asses to the app on a device. And gets displayed when one minimises the app for x-minutes, or exits the app.vThus if one logs out, and logs back into the app with just PIN, raises security concerns (where they really logged out).
Proposal: We could change login with password to login with PIN. That way, when one logs out, and reselects their username from the list, they just enter PIN.
Alternative(the better solution): We have a CTA in the dropdown list which allows one to add multiple users. Anyone who logs out of the app, isn't shown in the list. Whoever logs in and only exits that app without logging out, will still be able to access their account with just PIN.

Cc: @rowo @LZRS @DebbieArita


various improvements to the packet number generator

Fixes #398.
We skip packet numbers to defend against the Optimistic ACK attack.

Reduce calls to crypto/rand.Read. Instead, use a math/rand.Rand, initialized with cryptographic random.
Only skip packet numbers in the application data packet number space. Initial and Handshake packets are only used for very few packets during the connection, so there was only a small chance we'd skip a PN there anyway.
Implement an exponential backoff for skipping packet numbers, starting at an expected frequency of once per 256 packets, going up to once per 128*1024 packets.

A good way to think about this is by looking at the packet numbers skipped per max congestion windows (10000 packets). The current algorithm skips (on average) one PN in 500, which leads to 20 skipped packet numbers per cwnd (and therefore, per RTT). As a consequence, the ACK frames will have roughly 20 ACK ranges (assuming no packet loss). Clearly, this is a waste.

Why isn't the optimistic ack attack an issue for long-lived connections? Sure, it requires more upfront investment from an attacker, but then they can still max out the server's connection "for free"?

It is, but we don't need a check every n packets. Note that we never stop skipping packet numbers, we just do it less and less frequently. By using exponential backoff, we make sure that the peer received about as many packets as it can expect to receive before the next skip as it has already successfully acknowledged. I believe that this is "good enough" defense against an attack that is not that costly on our side.

Using a mathematical PRNG seeded with cryptographic random means an attacker could (in principle) reconstruct the seed, and avoid the defense. Not an issue if we don't worry about long-lived connections.

Unfortunately, I don't have precise numbers here how many numbers you can receive from math/rand.Rand before it becomes feasible to reconstruct the internal state. My intuition is many, and I'm assuming that's good enough to make this attack impractical in practice.


Add support for ViewPager2

Current LiquidSwipeViewPager is extended from androidx.viewpager.widget.ViewPager.
Create LiquidSwipeViewPager which would extend androidx.viewpager2.widget.ViewPager2.

Closing since this is currently infeasible due to the following reasons,

androidx.viewpager2.widget.ViewPager2 is a final class and can't be extended.
There isn't a clean way to set scroller duration in ViewPager2 (to slow down the page transitions). (reference)
ViewPager2 doesn't support reverse drawing order when setting PageTransformer (reference), although this can be worked around by changing the op in the ClipPathProvider to Region.Op.DIFFERENCE and adjusting the PageTransformer to 'clip' the next view instead of the previous view. But it's not worth the hassle for most users considering that the essence of the animation would be lost if the scroller duration can't be increased.


[Documentation] fix a wrong link in dev/table/sql

In dev/table/sql.html there is link to an anchor on dev/table/common.html which currently points at:
https://ci.apache.org/projects/flink/flink-docs-master/dev/table/common.html#register-a-table-in-the-catalog
It is outdated, the right link is:
https://ci.apache.org/projects/flink/flink-docs-master/dev/table/common.html#register-tables-in-the-catalog
Release 1.4 has the same problem.

+1

merging.


Patch to fix PER encoding length of IEs

Added following:

improvements to handle PER encoding in S1AP.
included temporary stream Id fix from https://github.com/omec-project/openmme/pull/27
changes to make logging uniform (missing \n, dots, etc.)

retest this please

retest this please

retest this please


Upgrade to Az

In section "Upgrade to Az" it says that "Keep the AzureRM module installed with PowerShell 5.1 for Windows, but install the Az module for PowerShell Core 6.x or later.". Does that mean I can have both running on the same machine?

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

ID: 95685be3-b179-5ae8-b984-740bcbb007f4
Version Independent ID: 524598ca-76aa-c8f5-cde1-d6bd626b757b
Content: Introducing the Azure PowerShell Az module
Content Source: docs-conceptual/azps-3.5.0/new-azureps-module-az.md
GitHub Login: @sptramer
Microsoft Alias: sttramer

@ravick4u Thank you for the feedback!  We are investigating this and will get back to you shortly.

@ravick4u That is my understanding as there are some use cases for the legacy AzureRM, however this has understandably been an area of confusion for a lot of people so I am going to ask the document author to confirm.
@sptramer Would you mind confirming both dual installation scenarios as well as full migration?
https://docs.microsoft.com/en-us/powershell/azure/new-azureps-module-az?view=azps-3.6.1#upgrade-to-az
Dual Installation
(in scenarios where legacy dependencies on AzureRM exist)

To keep up with the latest Azure features in PowerShell, you should migrate to the Az module as soon as possible. If you're not ready to install the Az module as a replacement for AzureRM, you have a couple of options available to experiment with Az:

Use a PowerShell environment with Azure Cloud Shell. Azure Cloud Shell is a browser-based shell environment which comes with the Az module installed and Enable-AzureRM compatibility aliases enabled.

Keep the AzureRM module installed with PowerShell 5.1 for Windows, but install the Az module for PowerShell Core 6.x or later. PowerShell 5.1 for Windows and PowerShell Core use separate collections of modules. Follow the instructions to install PowerShell Core and then install the Az module from a PowerShell Core terminal.

Upgrade/Migrate to Az
To upgrade from an existing AzureRM install:

Uninstall the Azure PowerShell AzureRM module
Install the Azure PowerShell Az module
OPTIONAL: Enable compatibility mode to add aliases for AzureRM cmdlets with Enable-AzureRMAlias while you become familiar with the new command set. See the next section or Start migration from AzureRM to Az for more details.

@ravick4u As long as the AzureRM module is installed with PowerShell 5.1 for Windows and the Az module is installed on PowerShell Core 6.x or later, you can have both versions.
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


[New Feature] If  GraphScope website need AOS animations for enhanced scroll effects .I can try to pull request

Add AOS animations for https://graphscope.io/
Applied animations to various sections of the site
Improved user experience with smooth scroll effects
just like:
https://streampark.apache.org/zh-CN/
https://michalsnik.github.io/aos/

Thanks for your proposal, @LofiSu!
Since graphscope.io serves as a project introduction website, we aim to make it informative and may restraint on the animation effects.
However, if you are interested in contributing to GraphScope project, you are welcome to make other PRs.
or more specifically, if you want to participating on the front-end development, you may find many todos here:
https://github.com/GraphScope/portal .
Thanks in advance and again!


LocalParticipant: Set server track mute status based on local track

Instead of changing the local track mute status when the server status
does not match, update the server's status to the local state. This
prevents a local track from getting unmuted without the user's action
when the states do not match.
Fixes #28

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Though the original issue I was having is fixed with other server & client changes, I still think that this change or something like it is important. For mute status, the local client should be the source of truth to avoid a user's tracks from getting unmuted without their knowledge or consent.

@bekriebel I'm closing this ticket since your needs are resolved by the server-side fix. lmk if you are having any other issues.


Date Picker Translation

Hi Team,
We want to translate the Date range filter in different languages. Do you have any functionality or configuration which can achieve this or we need to write a custom JS logic for this?
Thanks
Lovekesh Goel

The widget is provided by https://fomantic-ui.com
I'd check that project of to see if it's possible.

Hi David,
Please let me know your conclusion and if you need anything from my side.
Thanks
Lovekesh Goel

Localizing the Fomantic-UI date picker widget involves providing translations for text and format settings so that the widget displays dates, months, and instructions according to a specific locale's standards. Fomantic-UI, being a community fork of Semantic-UI, extends and enhances the functionalities offered by Semantic-UI, including localization support.
To localize the Fomantic-UI date picker widget, you typically need to adjust settings for the calendar module, which is responsible for date picking functionalities. Here's a step-by-step guide to do so:
1. Include Fomantic-UI in Your Project
Ensure that Fomantic-UI is included in your project. If you haven't done so, you can include it via CDN or install it using npm or yarn.
2. Add Localization Files
Fomantic-UI provides localization files for various components, including the calendar (date picker). These files are located in the dist/components/ directory within the Fomantic-UI package. Make sure you include the localization file for your desired locale in your project.
3. Initialize the Calendar with Localized Settings
When initializing the calendar widget, you can specify the localization settings directly in the initialization options or use a predefined localization file. Here's an example of how to manually set localization for a French locale:
$('#yourDatepickerId').calendar({
  text: {
    days: ['Dimanche', 'Lundi', 'Mardi', 'Mercredi', 'Jeudi', 'Vendredi', 'Samedi'],
    months: ['Janvier', 'Février', 'Mars', 'Avril', 'Mai', 'Juin', 'Juillet', 'Août', 'Septembre', 'Octobre', 'Novembre', 'Décembre'],
    monthsShort: ['Jan', 'Fév', 'Mar', 'Avr', 'Mai', 'Jun', 'Jul', 'Aoû', 'Sep', 'Oct', 'Nov', 'Déc'],
    today: 'Aujourd\'hui',
    now: 'Maintenant',
    am: 'AM',
    pm: 'PM'
  },
  formatter: {
    date: function (date, settings) {
      if (!date) return '';
      var day = date.getDate();
      var month = date.getMonth() + 1;
      var year = date.getFullYear();
      return day + '/' + month + '/' + year;
    }
  }
});

4. Test Your Localization
After setting the localization options, test the date picker to ensure that it displays information according to the specified locale. Check days, months, and any textual content like "Today" or "Now" to confirm they are correctly localized.
5. Adjust According to Your Locale
Different locales might have specific requirements, such as the first day of the week being Monday instead of Sunday, or using different date formats. Adjust the firstDayOfWeek and formatter options as needed to meet these locale-specific requirements.
By following these steps, you can effectively localize the Fomantic-UI date picker widget for your web application, ensuring a better user experience for users from different locales.


Issue #SB-29116 fix: RC download certificate through CSL added

Issue #SB-29116 fix: RC download certificate through CSL added


Document Filter syntax, and create filter debugger

After discussing with @stevvooe (related to https://github.com/moby/moby/issues/35891, https://github.com/moby/moby/pull/35896);

I should also build a filter debugger or make some docs around filters
There are some sharp edges to using filters that are there to make “everything” possible and I’ll make sure their covered

This syntax is documented in the godoc for filters.


Celery 3.0.9 AttributeError: 'NoneType' object has no attribute 'conf'

I am getting the following error after installing Celery==3.0.9 and trying to run ./manage.py celery worker
...
/local.py", line 169, in __getattr__
    return getattr(self._get_current_object(), name)
AttributeError: 'NoneType' object has no attribute 'conf'

Hi all,
I am using oracle-soa-plugin-1.3.1.xldp plugin and when i try to run sun soaDeploy.py and it shows me the attribute error.
runScriptFromDaemon("/tmp/ot-20151002T004427254/soaDeploy.py")
Disconnected from weblogic server: AdminServer
Processing [HelloWorld]
Traceback (most recent call last):
File "/tmp/ot-20151002T004428123/daemon.py", line 350, in ?
exec command
File "", line 1, in ?
File "/tmp/ot-20151002T004428123/daemon.py", line 277, in runScriptFromDaemon
execfile(scriptPath)
File "/tmp/ot-20151002T004427254/soaDeploy.py", line 497, in ?
managedServerUrl="t3://"+deployed.container.host.address+":"+str(deployed.container.port)
AttributeError: 'NoneType' object has no attribute 'address'
Could anyone help me how to avoid this error please.


List of ID's not mapped correctly with fluentapi

Describe the bug
When using the fluent api to add an argument that should hold a list of id's, the original type is not remembered because of the rewrite happening in RelayIdFieldExtensions.RewriteInputFieldType
The AddSerializerToInputField does check for parametertype in
Type resultType = definition.Parameter?.ParameterType ?? ((ClrTypeReference)definition.Type).Type;
But parameter is not set when using fluent api. If I set a dummy parameterinfo myself, it works as expected
To Reproduce
Steps to reproduce the behavior:
Add a parameter using fluentapi:
mytype.Field("getStuffById")
                    .Type<ListType<Stuff>>()
                    .Argument("ids", d => {
                        d.Type(typeof(int[])).ID("Stuff");
                    })
                    .ResolveWith<StuffResolver<T>>(resolver=> resolver.GetByIdsAsync(default!, default!, default));

Expected behavior
To be able to call getStuffById with a list of ID's -> when I do this an exception is thrown that int cannot be added to a list of strings, this is because FieldValueSerializer is initialized with IdType instead of int[]
Additional context
Workaround:
public class DummyParameterInfo: ParameterInfo
    {
        private Type type;

        public DummyParameterInfo(Type type)
        {
            this.type = type;
        }

        public override Type ParameterType => type;
    }

....
mytype.Field("getStuffById")
                    .Type<ListType<Stuff>>()
                    .Argument("ids", d => {
                        d.Extend().OnBeforeCreate(def => def.Parameter = new DummyParameterInfo(typeof(int[])));
                        d.Type(typeof(int[])).ID("Stuff");
                    })
                    .ResolveWith<StuffResolver<T>>(resolver=> resolver.GetByIdsAsync(default!, default!, default));

why do you specify the type as d.Type(typeof(int[]))
better would be here to leave that out or specify the schema type.

Which version are you using?

why do you specify the type as d.Type(typeof(int[]))
better would be here to leave that out or specify the schema type.

Leaving it out crashes in RelayIdFieldExtensions.RewriteInputFieldType because it has nothing to rewrite there, it can't detect the type from anything since with fluent you always have to specify it (or so I thought?)
I also tried d.Type<ListType>().ID("Stuff") instead of the clrtype, but it has the same issue.
I'm using v10.5.3, but if I look at the current develop branch,
https://github.com/ChilliCream/hotchocolate/blob/develop/src/HotChocolate/Core/src/Types/Types/Relay/Extensions/RelayIdFieldExtensions.cs still redirects the definition.Type  to idType, loosing the given type with d.Type()
So else if (definition.Type is ExtendedTypeReference typeReference) at https://github.com/ChilliCream/hotchocolate/blob/8299a944864a07bc0881ef06ce68818d801533d2/src/HotChocolate/Core/src/Types/Types/Relay/Extensions/RelayIdFieldExtensions.cs#L173 will always get IdType for definition.Type


Problem at api = keepa.Api(accesskey)

Hi,
Thanks for making the software available. Python 3.7.3 (v3.7.3:ef4ec6ed12, Mar 25 2019, 21:26:53) [MSC v.1916 32 bit (Intel)] on win32
Windows 10 32bit
on a HyperV virtual machine
Same error on my Windows 10 64bit machine
I install the module using “pip install keepa”
Running this code in sublime3:
import keepa
accesskey = ‘6999999999999999999999999’
api = keepa.Keepa(accesskey)
products = api.query(‘B0088PUEPK’) # returns list of product data #B07D3DTW4G

I get: AttributeError: module ‘keepa’ has no attribute ‘Keepa’
i also tried the same code with the following in place of api = keepa.Keepa(accesskey)
api = keepa.Api(accesskey)
Same error.
Any help would be appreciated.
Thanks

What version of keepa are you using?

I just downloaded the module yesterday. Version 0, 18, 0

Can you run the following?
from keepa.interface import Keepa

I get
ModuleNotFoundError: No module named 'keepa.interface'; 'keepa' is not a package

Is there an earlier version I could use in the meantime?
Sent from my iPhone

On Jul 22, 2019, at 12:16 PM, Alex Kaszynski<EMAIL_ADDRESS>wrote:
Can you run the following?
from keepa.interface import Keepa
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I can't reproduce this locally.  I recommend you try cloning this repo and installing from the source.  There's likely a problem with your install.  When you install it, use the -e development flag.

Cloning repo? Do you mean renaming the package directory?
On Tue, Jul 23, 2019 at 7:49 AM Alex Kaszynski<EMAIL_ADDRESS>wrote:

I can't reproduce this locally. I recommend you try cloning this repo and
installing from the source. There's likely a problem with your install.
When you install it, use the -e development flag.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/akaszynski/keepa/issues/41?email_source=notifications&email_token=AKOL6OHTJOEP3RY4KMN6XSDQA3V2XA5CNFSM4IFTSVQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2S3L3Y#issuecomment-514176495,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKOL6OB5FKPIPQKZZOKC4KDQA3V2XANCNFSM4IFTSVQQ
.

You can either click the download button on the main page of this git website, or run:
$ git clone https://github.com/akaszynski/keepa
$ cd keepa
$ pip install -e .

Looks like you're using windows.  You can get Git for Windows at https://git-scm.com/download/win

I ran pip install -e and got this.

PS C:\Windows\system32\keepa> pip install -e
Usage:
pip install [options]  [package-index-options] ...
pip install [options] -r  [package-index-options] ...
pip install [options] [-e]  ...
pip install [options] [-e]  ...
pip install [options] <archive url/path> ...
-e option requires 1 argument

Looks like you forgot the period.
pip install -e .

Got it. It worked. What do I do next?

Do you encounter the same problem when importing keepa?

Yes.
AttributeError: module 'keepa' has no attribute 'Keepa'

It not when I import keepa. The error occurs at
api = keepa.Keepa(accesskey)

Please run this and let me know what it outputs.
import keepa
print(keepa.__file__)

C:\inetpub\wwwroot\keepa.py

That is the location and name of the script I running.

I change the name of my script and now it works.

Glad we figured it out.

Can I ask a question here?

Please email me at<EMAIL_ADDRESS>

I figured it out. Thanks for your help.
Anthony Brea
On Thu, Jul 25, 2019 8:26 AM, Alex Kaszynski<EMAIL_ADDRESS>wrote:
Please email me at<EMAIL_ADDRESS>—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[pruner] keep state of last manually processed SST files

PR introduces state tracking for the last manually processed SST files in memory. This prevents the recurrence of situations where the same file is repeatedly selected for manual compaction because its initial compaction run was a noop.
Cherry pick is required to unblock partners full node.

@phoenix-o Does this have to be part of the v1.17.0 release next week?

@ebmifa: ideally, yes. The fix is requested by Bybit


Description
Adds a new field in the OdhDashboardConfig called groupsConfig. Removes logic dealing with configmaps and gets groups directly based on what is in the CR. Replaces logic and types to accomodate this change.
The PR additionally disables the notebook controller page if the user is not in the allowed groups to resolve https://github.com/opendatahub-io/odh-dashboard/issues/342
How Has This Been Tested?

Modify adminGroups in the groupsConfig section of the CR
Verify relevant admin permissions are applied when accesing the dashboard route.
Modify allowedGroups.
Verify access to the notebook controller

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

@DaoDaoNoCode @samuelvl I checked the code and the admin permissions work correctly, but I found no mention of use for allowedGroups, is it used externally?

@maroroman allowedGroups is related with https://github.com/opendatahub-io/odh-dashboard/issues/342
Only the people in this group should be allowed to use the jupyter spawner UI, otherwise they should get a 403.

/hold This PR introduces some breaking changes (configmaps logic is replaced by CR). Targeting RHODS 1.16.

I've inquired details on the ticket about the direction this PR is taking. I'm a bit unsure of the approach of using the ODH Dashboard Config as a dumping ground of everything -- it seems a bit brute force and short sighted for the growth of the Dashboard.

From a user perspective it's a little bit cumbersome. ConfigMaps are easily accessible. If this config in the ODH Dashboard Config, from the OCP Console I have to go down to Administration, filter/look for OdhDashbordConfig, Instance, and then finally edit it... That's a long path to add admin or allowed groups...


sudo: 1.9.5p2 -> 1.9.6

Motivation for this change
Update
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[X] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 116423 at c18933f4 run on x86_64-linux 1

1 package skipped due to time constraints:

mysql-workbench

7 packages built successfully:

charliecloud
interlock
lxqt.lxqt-sudo
passExtensions.pass-tomb
qsudo
sudo
tomb

1 suggestion:

error: no-flags-array
configureFlagsArray is only intended to be used in Bash, not as a Nix attribute.
Near pkgs/tools/security/sudo/default.nix:45:3:
   |
45 |   configureFlagsArray = [
   |   ^


Add health URL when in maintenance mode

Mesos/Marathon still needs to be able to check the health of the app.

👍


[R-package] [docs] remove devtools dependency for docs builds (fixes #3036)

I'd like to propose removing the dependency on devtools in our readthedocs builds. We are only using it for devtools::document(), which is just a thin wrapper around roxygen2::roxygenize().
devtools is a very expensive dependency (it depends recursively on successfully building 101 other R packages) compared to roxygen2 (41 recursive dependencies), and we're already installing roxygen2 anyway.
I'm also proposing bumping up our version of roxygen2 to 7.1.0, the most recent version used in DESCRIPTION
https://github.com/microsoft/LightGBM/blob/05d89a1a1f48a6096e3d2342a517e048d6765bdd/R-package/DESCRIPTION#L44
How I got dependency counts
I figured out those recursive dependency numbers with the pkgnet package, a project I work on with some friends from a previous job. In an R environment, you can run pkgnet::CreatePackageReport('devtools') (for example), to see a summary of the dependencies.

@StrikerRUS I created this as a LightGBM branch, not on my fork. Could you enable it on readthedocs so we can test?

@jameslamb

Could you enable it on readthedocs so we can test?

Sure, done!

@StrikerRUS hey it looks like this is working!

build: https://readthedocs.org/projects/lightgbm/builds/11019657/
docs: https://lightgbm.readthedocs.io/en/fix-remove-devtools/

Some notes:

I had to replace devtools::document() with roxygen2::roxygenize(load='installed') (not just roxygen2::roxygenize()) because of  https://github.com/r-lib/roxygen2/issues/822#issuecomment-533183162
given ^, it's important that conda's R knows to look for lightgbm in the same place it was installed by build_r.R. That's why I added export R_LIBS...
the first few builds were very tough to debug on https://readthedocs.org/projects/lightgbm/builds/ because the logs from build_r.R were  disabled by the sed call. Can we please remove it so the logs can be used for debugging? They aren't THAT long, and I think they're valuable

By "it's working", I mean updating pkgdown and roxygen2 in the conda environment. I still need too add the use of pkgdown url option for #3036. Trying that next.

The changes for #3036 do not seem to be working :/

build: https://readthedocs.org/projects/lightgbm/builds/11019779/
docs: https://lightgbm.readthedocs.io/en/fix-remove-devtools/R/reference/

The GitHub logo was updated correctly, but links like Source: R/lightgbm.R point to the wrong place.
For example, Source: R/lightgbm.R on https://lightgbm.readthedocs.io/en/fix-remove-devtools/R/reference/agaricus.train.html points to https://github.com/Microsoft/LightGBM/blob/master/R/lightgbm.R
I'm investigating

Ok I think this is fully working now and closes #3036 . This is ready for review.

the first few builds were very tough to debug on https://readthedocs.org/projects/lightgbm/builds/ because the logs from build_r.R were disabled by the sed call. Can we please remove it so the logs can be used for debugging? They aren't THAT long, and I think they're valuable

Are you speaking about this sed?
https://github.com/microsoft/LightGBM/blob/b60294b81df95c6735285ff76242c448757925e4/docs/conf.py#L243
This sed was used to prevent double building of LightGBM similarly to trick for CI with R cmd check and allows us to fit in RTD build time limits.
You can safely drop ca-certificates dependency now.
In build_reference() document argument is replaced by devel since 1.4.0 version.
https://github.com/r-lib/pkgdown/blob/c181e5c9ac09b28974a97f5e32deddc36690fcf2/R/build-reference.R#L139
Refer to https://github.com/r-lib/pkgdown/releases/tag/v1.4.0

build_site(), build_reference() and build_home() gain a parameter
devel which controls whether you're in deployment or development mode.
It generalises and replaces (with deprecation) the existing document
argument.

the first few builds were very tough to debug on https://readthedocs.org/projects/lightgbm/builds/ because the logs from build_r.R were disabled by the sed call. Can we please remove it so the logs can be used for debugging? They aren't THAT long, and I think they're valuable

Are you speaking about this sed?
https://github.com/microsoft/LightGBM/blob/b60294b81df95c6735285ff76242c448757925e4/docs/conf.py#L243
This sed was used to prevent double building of LightGBM similarly to trick for CI with R cmd check and allows us to fit in RTD build time limits.
You can safely drop ca-certificates dependency now.
In build_reference() document argument is replaced by devel since 1.4.0 version.
https://github.com/r-lib/pkgdown/blob/c181e5c9ac09b28974a97f5e32deddc36690fcf2/R/build-reference.R#L139
Refer to https://github.com/r-lib/pkgdown/releases/tag/v1.4.0

build_site(), build_reference() and build_home() gain a parameter
devel which controls whether you're in deployment or development mode.
It generalises and replaces (with deprecation) the existing document
argument.

Oh ok! Yes I misunderstood what that sed command was doing, thanks for the clarification. But I still think we should remove it. roxygen2::roxygenize(load = "installed") will say "don't try to rebuild the package, just assume it has already been built and installed".
I'll try removing ca-certificates and updating build_reference() call tonight.

Seems that removing ca-certificates worked!

build: https://readthedocs.org/projects/lightgbm/builds/11027879/
docs: https://lightgbm.readthedocs.io/en/fix-remove-devtools/R/reference/agaricus.train.html

close-reopen to re-trigger CI. Azure got stuck trying to download MiKTeX :grimacing:


PingPerf thread Checkpoint failed on JDK17 linux aarch64

01:02:21.458  "/internaljdk/java/openjdk/bin/java" -version
01:02:21.458  java version "17.0.8-beta" 2023-07-18
01:02:21.458  IBM Semeru Runtime Certified Edition 17.0.8+6-202306171433 (build 17.0.8-beta+6-202306171433)
01:02:21.458  Eclipse OpenJ9 VM 17.0.8+6-202306171433 (build master-a09876e49, JRE 17 Linux aarch64-64-Bit Compressed References 20230617_423 (JIT enabled, AOT enabled)
01:02:21.458  OpenJ9   - a09876e49
01:02:21.458  OMR      - e09e7b5f1
01:02:21.458  JCL      - c723411f39 based on jdk-17.0.8+6)

On ub22, thread Checkpoint failed: Internal build
00:59:10.821  Launching defaultServer (Open Liberty <IP_ADDRESS>-beta/wlp-1.0.78.cl230620230612-1100) on Eclipse OpenJ9 VM, version 17.0.7+7 (en_US)
00:59:10.821  CWWKE0953W: This version of Open Liberty is an unsupported early release version.
00:59:16.292  [AUDIT   ] CWWKE0001I: The server defaultServer has been launched.
00:59:41.233  [AUDIT   ] CWWKG0093A: Processing configuration drop-ins resource: /opt/ol/wlp/usr/servers/defaultServer/configDropins/defaults/keystore.xml
00:59:41.233  [AUDIT   ] CWWKG0093A: Processing configuration drop-ins resource: /opt/ol/wlp/usr/servers/defaultServer/configDropins/defaults/open-default-port.xml
01:00:06.032  [ERROR   ] CWWKF0001E: A feature definition could not be found for checkpoint-1.0
01:00:14.358  [AUDIT   ] CWWKZ0058I: Monitoring dropins for applications.
01:00:18.272  [WARNING ] CWOWB1009W: Implicit bean archives are disabled.
01:01:08.491  [AUDIT   ] CWWKC0451I: A server checkpoint "afterAppStart" was requested. When the checkpoint completes, the server stops.
01:01:08.491  [AUDIT   ] CWWKZ0022W: Application pingperf has not started in 30.028 seconds.
01:01:08.491  [ERROR   ] CWWKC0453E: The server checkpoint request failed with the following message: CWWKC0457E: An error occurred preparing to take a checkpoint with the following message: CWWKZ0022W: Application pingperf has not started in 29.948 seconds.
01:01:08.491  [AUDIT   ] CWWKE0084I: The server defaultServer is stopping because thread Checkpoint failed, exiting... (0000005a) called the method java.lang.System.exit:
01:01:08.491  	at java.base/java.lang.System.exit(System.java:502)
01:01:08.491  	at io.openliberty.checkpoint.internal.CheckpointImpl.lambda$checkpointOrExitOnFailure$1(CheckpointImpl.java:266)
01:01:08.491  	at java.base/java.lang.Thread.run(Thread.java:857)
01:01:08.491
01:01:08.491  [AUDIT   ] CWWKF0012I: The server installed the following features: [cdi-3.0, concurrent-2.0, jndi-1.0, jsonp-2.0, restfulWS-3.0, restfulWSClient-3.0, servlet-5.0].
01:01:08.491  [AUDIT   ] CWWKF0011I: The defaultServer server is ready to run a smarter planet. The defaultServer server started in 131.953 seconds.
01:01:08.491  [AUDIT   ] CWWKE1100I: Waiting for up to 30 seconds for the server to quiesce.
01:01:28.952  [AUDIT   ] CWWKZ0001I: Application pingperf started in 54.948 seconds.
01:01:29.368  [AUDIT   ] CWWKZ0009I: The application pingperf has stopped successfully.
01:01:36.059  -----------------------------------
01:01:36.059  criu_pingPerf_testCreateRestoreImageAndPushToRegistry_0_FAILED
01:01:36.059  -----------------------------------

On rhel9, exec container process /bin/sh: Exec format error: internal build
00:23:02.336  [2/2] STEP 11/23: RUN set -eux;     ARCH="$(uname -m)";     case "${ARCH}" in        aarch64|arm64)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_aarch64';          DUMB_INIT_SHA256=b7d648f97154a99c539b63c55979cd29f005f88430fb383007fe3458340b795e;          ;;        amd64|x86_64)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_x86_64';          DUMB_INIT_SHA256=e874b55f3279ca41415d290c512a7ba9d08f98041b28ae7c2acb19a545f1c4df;          ;;        ppc64el|ppc64le)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_ppc64le';          DUMB_INIT_SHA256=3d15e80e29f0f4fa1fc686b00613a2220bc37e83a35283d4b4cca1fbd0a5609f;          ;;        s390x)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_s390x';          DUMB_INIT_SHA256=47e4601b152fc6dcb1891e66c30ecc62a2939fd7ffd1515a7c30f281cfec53b7;          ;;       *)          echo "Unsupported arch: ${ARCH}";          exit 1;          ;;     esac;     curl -LfsSo /usr/bin/dumb-init ${DUMB_INIT_URL};     echo "${DUMB_INIT_SHA256} */usr/bin/dumb-init" | sha256sum -c -;     chmod +x /usr/bin/dumb-init;
00:23:02.336  exec container process `/bin/sh`: Exec format error
00:23:02.337  Error: building at STEP "RUN set -eux;     ARCH="$(uname -m)";     case "${ARCH}" in        aarch64|arm64)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_aarch64';          DUMB_INIT_SHA256=b7d648f97154a99c539b63c55979cd29f005f88430fb383007fe3458340b795e;          ;;        amd64|x86_64)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_x86_64';          DUMB_INIT_SHA256=e874b55f3279ca41415d290c512a7ba9d08f98041b28ae7c2acb19a545f1c4df;          ;;        ppc64el|ppc64le)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_ppc64le';          DUMB_INIT_SHA256=3d15e80e29f0f4fa1fc686b00613a2220bc37e83a35283d4b4cca1fbd0a5609f;          ;;        s390x)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_s390x';          DUMB_INIT_SHA256=47e4601b152fc6dcb1891e66c30ecc62a2939fd7ffd1515a7c30f281cfec53b7;          ;;       *)          echo "Unsupported arch: ${ARCH}";          exit 1;          ;;     esac;     curl -LfsSo /usr/bin/dumb-init ${DUMB_INIT_URL};     echo "${DUMB_INIT_SHA256} */usr/bin/dumb-init" | sha256sum -c -;     chmod +x /usr/bin/dumb-init;": while running runtime: exit status 1
00:23:02.337  -----------------------------------
00:23:02.337  criu_pingPerf_testCreateRestoreImageAndPushToRegistry_0_FAILED
00:23:02.337  -----------------------------------

Noticed the same issue on JDK11 and JDK17.

Failed build -  Using dockerfile from OpenLiberty/ci.docker repo branch instanton with commit hash 3fbc2789ee736701f729febb747082ff9cbbd170
[Passed build on JDK11 linux aarch64] last week (https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder_imageUpload/77/consoleFull) - Using dockerfile from OpenLiberty/ci.docker repo branch instanton with commit hash ee87dfa7f7c7de01d12786aa71517fa8f4007883
Passed on JDK11 xlinux - Using dockerfile from OpenLiberty/ci.docker repo branch instanton with commit hash 3fbc2789ee736701f729febb747082ff9cbbd170 , which the same sha as the one that has issues above

https://github.com/OpenLiberty/ci.docker/compare/ee87dfa7f7c7de01d12786aa71517fa8f4007883...3fbc2789ee736701f729febb747082ff9cbbd170
We use releases/latest/beta/Dockerfile.ubi.openjdk17 from https://github.com/OpenLiberty/ci.docker.git instanton branch

Trying to narrow down the issue in the Open liberty repo. The test can be passed with 30caf5d0e1d71cc9efefc3bec250a6c72c084168 (Grinder link). Grinder failed with 38c8145efc63adc3686466ef4a3322c20720330d (Grinder link)
00:01:40.541  create restore image ol-instanton-test-pingperf-restore ...
00:01:54.907  Performing checkpoint --at=afterAppStart
00:01:55.324
00:03:21.531  CWWKE0954E: The specified (afterappstart) checkpoint phase is empty or unknown.
00:03:21.531
00:03:22.538  -----------------------------------
00:03:22.538  criu_pingPerf_testCreateRestoreImageAndPushToRegistry_0_FAILED

The PingPerf test is excluded temporarily on zlinux and alinux.

Also noticed exec /bin/sh: exec format error on plinux and zlinux
00:08:13.437  [2/2] STEP 10/23: COPY fixes/ /opt/ol/fixes/
00:08:14.706  --> 7c4b2fa0f46
00:08:14.707  [2/2] STEP 11/23: RUN set -eux;     ARCH="$(uname -m)";     case "${ARCH}" in        aarch64|arm64)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_aarch64';          DUMB_INIT_SHA256=b7d648f97154a99c539b63c55979cd29f005f88430fb383007fe3458340b795e;          ;;        amd64|x86_64)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_x86_64';          DUMB_INIT_SHA256=e874b55f3279ca41415d290c512a7ba9d08f98041b28ae7c2acb19a545f1c4df;          ;;        ppc64el|ppc64le)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_ppc64le';          DUMB_INIT_SHA256=3d15e80e29f0f4fa1fc686b00613a2220bc37e83a35283d4b4cca1fbd0a5609f;          ;;        s390x)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_s390x';          DUMB_INIT_SHA256=47e4601b152fc6dcb1891e66c30ecc62a2939fd7ffd1515a7c30f281cfec53b7;          ;;       *)          echo "Unsupported arch: ${ARCH}";          exit 1;          ;;     esac;     curl -LfsSo /usr/bin/dumb-init ${DUMB_INIT_URL};     echo "${DUMB_INIT_SHA256} */usr/bin/dumb-init" | sha256sum -c -;     chmod +x /usr/bin/dumb-init;
00:08:15.077  exec /bin/sh: exec format error
00:08:17.707  Error: building at STEP "RUN set -eux;     ARCH="$(uname -m)";     case "${ARCH}" in        aarch64|arm64)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_aarch64';          DUMB_INIT_SHA256=b7d648f97154a99c539b63c55979cd29f005f88430fb383007fe3458340b795e;          ;;        amd64|x86_64)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_x86_64';          DUMB_INIT_SHA256=e874b55f3279ca41415d290c512a7ba9d08f98041b28ae7c2acb19a545f1c4df;          ;;        ppc64el|ppc64le)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_ppc64le';          DUMB_INIT_SHA256=3d15e80e29f0f4fa1fc686b00613a2220bc37e83a35283d4b4cca1fbd0a5609f;          ;;        s390x)          DUMB_INIT_URL='https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_s390x';          DUMB_INIT_SHA256=47e4601b152fc6dcb1891e66c30ecc62a2939fd7ffd1515a7c30f281cfec53b7;          ;;       *)          echo "Unsupported arch: ${ARCH}";          exit 1;          ;;     esac;     curl -LfsSo /usr/bin/dumb-init ${DUMB_INIT_URL};     echo "${DUMB_INIT_SHA256} */usr/bin/dumb-init" | sha256sum -c -;     chmod +x /usr/bin/dumb-init;": while running runtime: exit status 1
00:08:17.708  -----------------------------------
00:08:17.708  criu_pingPerf_testCreateRestoreImageAndPushToRegistry_0_FAILED

Grinder

Re: exec /bin/sh: exec format error the test seems to first build a Semeru image for the current platform and tags it as:
[2023-06-17T15:30:54.952Z] Successfully tagged localhost/local-ibm-semeru-runtimes:latest
[2023-06-17T15:30:55.393Z] 5f81856e35c2f07dde8b6df5734367f3583f5ce531d257cf50ce63011e926575

but when building the Liberty image we pull the amd64 image. I'm guessing we want to build the Liberty image on top of the local Semeru image, so we need to modify the Liberty image Dockerfile FROM line.

The ub22 thread Checkpoint failed is the same problem, but it looks different because somehow the Liberty image is successfully built because the x86-64 binaries are able to run on aarch64:
[2023-06-17T15:28:21.602Z] ++ uname -m
[2023-06-17T15:28:21.602Z] + ARCH=x86_64
[2023-06-17T15:28:21.602Z] + case "${ARCH}" in
[2023-06-17T15:28:21.602Z] + DUMB_INIT_URL=https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_x86_64
[2023-06-17T15:28:21.602Z] + DUMB_INIT_SHA256=e874b55f3279ca41415d290c512a7ba9d08f98041b28ae7c2acb19a545f1c4df
[2023-06-17T15:28:21.602Z] + curl -LfsSo /usr/bin/dumb-init https://github.com/Yelp/dumb-init/releases/download/v1.2.5/dumb-init_1.2.5_x86_64

I don't know how Docker on aarch64 works and how it can do the above, but once the image is built and we try to start the server it fails.

Details about PingPerf tests:

it uses Semeru docker file: https://github.com/ibmruntimes/semeru-containers/blob/ibm/11-ea/jdk/ubi/ubi8/Dockerfile.open.releases.full
it uses the Liberty image Dockerfile from instanton branch, not main branch: https://github.com/OpenLiberty/ci.docker/blob/instanton/releases/latest/beta/Dockerfile.ubi.openjdk17
it modifies the Liberty image Dockerfile FROM line to pick up the local Semeru image: https://github.com/adoptium/aqa-tests/blob/165701fe12cfd961db19d2e130e5ca6f20ae79bf/external/criu/pingPerf.sh#L97
it changes the SDK to use the one from our nightly/weekly build

I think this is due to a recent Liberty change - switch to multi-stage builds https://github.com/OpenLiberty/ci.docker/commit/7aaf9c52f4aff99cf850d5fd37f83293afc773ed
Liberty image Dockerfile has 2 FROM
https://github.com/OpenLiberty/ci.docker/blob/instanton/releases/latest/beta/Dockerfile.ubi.openjdk17#L1
https://github.com/OpenLiberty/ci.docker/blob/instanton/releases/latest/beta/Dockerfile.ubi.openjdk17#L27
Since we only replace the first FROM - icr.io/appcafe/ibm-semeru-runtimes:open-17-jdk-ubi, the second FROM will still pull icr.io/appcafe/ibm-semeru-runtimes:open-17-ea-jdk-ubi-amd64, which cause the test to fail.
@ymanton do you know why Liberty uses two Semeru docker images?

@ymanton do you know why Liberty uses two Semeru docker images?

At one point we were using the Semeru EA images for the Liberty beta images.  But now for https://github.com/OpenLiberty/ci.docker/pull/412 we want the Liberty UBI beta images based on icr.io/appcafe/ibm-semeru-runtimes:open-17-jdk-ubi but it looks like there is an issue with the second FROM.  It needs to be updated to use icr.io/appcafe/ibm-semeru-runtimes:open-17-jdk-ubi also.

Once the Liberty dockerfile is finalized, we will update to use releases version (not beta) : https://github.com/OpenLiberty/ci.docker/blob/6b1c9dc9395ada7006da9fd0ebdb485602be98ea/releases/latest/full/Dockerfile.ubi.openjdk11

https://github.com/OpenLiberty/ci.docker/pull/412 has been updated to use the correct FROM (not to use the EA semeru image).

01:00:06.032  [ERROR   ] CWWKF0001E: A feature definition could not be found for checkpoint-1.0

Do you still see this error?  Before we auto configured this feature in our beta images.  But we should no longer be doing that.  Does your server.xml file you are using configure checkpoint-1.0 Liberty feature?  If so you can stop doing that once we have the Liberty GA for InstantOn.  We removed the need for this feature for Liberty InstantOn GA.

Thanks @tjwatson. The PingPerf test passed with https://github.com/OpenLiberty/ci.docker/pull/412.
Grinder
Grinder

Hi @tjwatson , I noticed that https://github.com/OpenLiberty/ci.docker/pull/412 is merged into main branch. When will it be ported into the instanton  branch? Or should we switch to using main branch: https://github.com/OpenLiberty/ci.docker/blob/main/releases/latest/beta/Dockerfile.ubi.openjdk17?

going forward use the main branch.  The instanton branch will be abandoned (maybe removed) at some point.

This issue is resolved. Thanks, everyone!


Consistent formatting with GNU indent

The options were chosen for similarity to dpcomp.c, with a maximum line length chosen from the rest:
indent -kr -l150 -nut -ts4 -brf -c1 *.c *.h

(GitHub's ?w=1 helps read this diff without noise from the 2-to-4 space indent change.)
(I plan to do some further refactoring, like introducing an enum for the LZ commands 0-7, and would prefer to avoid formatting changes mixed up with those or with algorithm improvements.)
Some manual changes were made afterwards:

In dpcomp.c, reverted these:
consider(..., (struct command) {
         ...,
         });

back to these:
consider(..., (struct command) {
    ...,
});

In proto.h, reverted this:
struct command {
    unsigned command:3;         // ...
    unsigned count:12;          // ...
    signed value:17;            // ...
};

back to this:
struct command {
    unsigned command: 3; // ...
    unsigned count:  12; // ...
    signed value:    17; // ...
};

In options.c, changed this:
struct options result = {.input = NULL,.output = NULL,.alignment = 0 };

to this:
struct options result = { .input = NULL, .output = NULL, .alignment = 0 };

In uncomp.c, changed two label:{s to label: {s

In proto.h, changed two single-line /* comments */ to // comments

Regarding further updates, I'm also curious if packing struct command into 32 bits is actually helping performance; maybe working with aligned integer fields would be worth a larger struct (particularly since space is all malloced at the outset due to the relatively small MAX_FILE_SIZE).


[Feat]: Adding Unlighthouse , regexr to useful tools

What feature?
I would like to add Unlighthouse and regexr to useful tools section
Unlighthouse generate google lighthouse reports for all routes at same time
Regexr is a site to test and build regular expressions
I would like to contribute this under GSSOC23
Add screenshots

Code of Conduct

[X] I agree to follow this project's Code of Conduct

You can add it only if the mentioned platforms are free to use.

You can add it only if the mentioned platforms are free to use.

Yes, they are free to use

thx for assigning me 😊


component build

Describe the bug
The s3 artifact URI in built recipe file does not reflect the bucket name, component name and component version configured in the gdk config file.
To Reproduce
In a directory, "hello", run  gdk component init -l python -t HelloWorld.
Edit gdk-config.json and replace the placeholders as shown below.
{
  "component": {
    "com.example.PythonHelloWorld": {
      "author": "Me",
      "version": "NEXT_PATCH",
      "build": {
        "build_system": "zip"
      },
      "publish": {
        "bucket": "gg-component-bucket",
        "region": "us-east-1"
      }
    }
  },
  "gdk_version": "1.0.0"
}

Edit recipe.yaml, change the name of the zip file to "hello.zip" as shown below

---
RecipeFormatVersion: "2020-01-25"
ComponentName: "{COMPONENT_NAME}"
ComponentVersion: "{COMPONENT_VERSION}"
ComponentDescription: "This is simple Hello World component written in Python."
ComponentPublisher: "{COMPONENT_AUTHOR}"
ComponentConfiguration:
  DefaultConfiguration:
    Message: "World"
Manifests:
  - Platform:
      os: all
    Artifacts:
      - URI: "s3://BUCKET_NAME/COMPONENT_NAME/COMPONENT_VERSION/hello.zip"
        Unarchive: ZIP
    Lifecycle:
      Run: "python3 -u {artifacts:decompressedPath}/HelloWorld/main.py {configuration:/Message}"

run gdk component build
Expected behavior
The artifact URI in greengrass-build/recipes/recipe.yaml reflects the bucket name, component name and component version configured in gdk-config.json.
Actual behavior
The artifact URI is "s3://BUCKET_NAME/COMPONENT_NAME/COMPONENT_VERSION/hello.zip"
Environment

OS:Darwin MacBook-Pro.local 21.2.0 Darwin Kernel Version 21.2.0: Sun Nov 28 20:28:54 PST 2021; root:xnu-8019.61.5~1/RELEASE_X86_64 x86_64
JDK version: NA
Nucleus version: NA

Additional context

Hi,
Artifacts URIs in the component recipe are updated with gdk component publish command only. The updated recipe with S3 artifact URIs is found under greengrass-build/recipes as <component-name>-<component-version>.yaml recipe file after the publish command is run.
So, the behavior you're seeing with gdk component build command is expected.

Oh I see. Thanks.
Sent from my iPhone

On Jan 20, 2022, at 5:18 PM, Saranya @.***> wrote:

Hi,
Artifacts URIs in the component recipe are updated with gdk component publish command only. The updated recipe with S3 artifact URIs is found under greengrass-build/recipes as -.yaml recipe file after the publish command is run.
So, the behavior you're seeing with gdk component build command is expected.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.


don't add close button to the modal-header when close.bind="false" is used

Thanks for the contribution.


Update IUC tools in Graph/Display data on .org

as stated in title
Installation sequence for tool-installers

[ ] Test using @galaxybot test this
[ ] Inspect CI output for expected changes
[ ] Deploy using @galaxybot deploy this if test install was successful
[ ] Merge this PR

@galaxybot test this

@galaxybot deploy this


Oracle DDL for View please extend the statetment with comment part

Hi,
can you please  to the DDL Statement for the View the comment part if the comment on is existing ?
You did it already for the table DDL :)
It would be very helpful.
Best regards
Denis

Reading DDL Statement for Views and Materialized views like in Oracle Tables implemented.

Verified
created issue #10154


Recursive map

Composite design pattern is a fact of programming especially in GUI.
When I was trying to contribute to Swift Package Manager, I had a problem which is we want to
resolve dynamic packages recursively, each package depends on other packages and so on.
https://github.com/apple/swift-package-manager/pull/1389
I think it would be helpful to make Swift support tree operations out of the box, here is the implementation of the first thing that could be needed,  recursiveMap
Maybe helpful in the UIView.subViews array in iOS.
I couldn't understand how to put unit tests here, but here is how I tested it in a side project using Xcode 10 and Swift 4.2
import XCTest
@testable import RecursiveMap

struct SimpleTreeNode {
    var name: String
    var children: [SimpleTreeNode]
}

class RecursiveMapTests: XCTestCase {

    func testSimpleTreeRecursiveMap() {
        let simpleTree: [SimpleTreeNode] = [
            SimpleTreeNode(name: "Root1", children: [
                SimpleTreeNode(name: "Child1", children: [
                    SimpleTreeNode(name: "Child1OfChild1", children: []),
                    SimpleTreeNode(name: "Child2OfChild1", children: []),
                    SimpleTreeNode(name: "Child3OfChild1", children: [])
                    ]),
                SimpleTreeNode(name: "Child2", children: []),
                SimpleTreeNode(name: "Child3", children: [
                    SimpleTreeNode(name: "Child1OfChild3", children: []),
                    SimpleTreeNode(name: "Child2OfChild3", children: [
                        SimpleTreeNode(name: "Child1OfChild2OfChild3", children: [])])
                    ])
            ])
        ]
        let array = simpleTree.recursiveMap({ $0.name }, { $0.children })
        XCTAssert(array.contains("Root1"))
        XCTAssert(array.contains("Child1"))
        XCTAssert(array.contains("Child1OfChild1"))
        XCTAssert(array.contains("Child2OfChild1"))
        XCTAssert(array.contains("Child3OfChild1"))
        XCTAssert(array.contains("Child2"))
        XCTAssert(array.contains("Child3"))
        XCTAssert(array.contains("Child1OfChild3"))
        XCTAssert(array.contains("Child2OfChild3"))
        XCTAssert(array.contains("Child1OfChild2OfChild3"))
        XCTAssertEqual(array.count, 10)
    }

}

We may need a lot of other tree operations, like recursiveForEach, recursiveReduce, recursiveFilter, recursiveSearch, etc...

Closing this PR due to inactivity — please feel free to reopen anytime! Note that a change like this requires a Swift Evolution proposal. If you'd like to discuss this proposed addition, please do so in the forums: https://forums.swift.org


Fixed lint issues

Fixed the lint issues. The return await statements are apparently redundant

Yeah, if a method is async or returns Promise you can omit the await.


assetpack folder .ab file cannot decompress   u3d asset

Expected behavior
/System.IO.IOException: Lz4 decompression error, write - 1 bytes but expect
bytes
Game/Sample
textures.zip
global-metadata.zip
libil2cpp.zip
Makeover Dream: Tile Match
https://github.com/ouyangwenyuan/OhMyFramework
ab packet hot update Framework
Framework
u3d cn version 2020.3.45f1c1
cant  decompress  u3d asset

BuildAssetBundleOptions.DeterministicAssetBundle

AssetBundle.SetAssetBundleDecryptKey()   key is  or not hook this

55ccde5a7cb81567

key is 55ccde5a7cb81567

{"Name":"dragonplus","Key":"35356363646535613763623831353637"} thanks 大佬


onOffsetChange firing when plugin loads

Looks like this fires when the plugin loads under the following conditions:
imageState.src passed into config without any other imageState options
ImageState.src and imageState.zoom passed in together
imageState.src and both imageState.offset.x and imageState.offset.y passed. If only one or the other offset property is passed, onOffsetChange does not fire on load
Seems to me this shouldn't fire unless you've manually changed the offset. Let me know what you think.

I think it could make sense both ways. When cropit is initiated with a preset offset, it is "changed" in the sense that it's different from the default.
Is this creating any issue for your application?

I think that is ok.
User of the plugin may have to manage by setting some flag. I am currently doing that. I manage a global flag, when plugin first loads, in onOffsetChange callback, that flag is checked, if it is false then change it to true and do nothing, if true then do normal processing.


Hi, there is possible to change the OHIF logo and banner in version 2.0?

:hand: We are a small team with limited resources. Your question is much more
likely to be answered if it is
a good question

Description
Questions can often be answered by our documentation. Unable to find an answer
in our docs? We'll try to help. In the meantime, if you answer your own
question, please respond with the answer here so that others may benefit as
well. Better yet, open a PR to expand our docs ^_^

Change OHIF Logo on top left : Whitelabeling

Prior to build, add the following white labeling code in Viewers>platform>viewer>public>config>default.js
Save your logo file in Viewers>platform>viewer>public>assets

window.config = {
.......

whiteLabeling: {

    createLogoComponentFn: function(React) {
      return React.createElement('a', {
        target: '_self',
        rel: 'noopener noreferrer',
        className: 'header-brand',
        href: '/',
        style: {
          display: 'block',
          textIndent: '-9999px',
          background: 'url(../assets/Your_Logo.png)',
          backgroundSize: 'contain',
          backgroundRepeat: 'no-repeat',
          width: '200px',
        },
      });
    },
  },
....
}

Thanks
I got this
window.config = {
// default: '/'
routerBasename: '/',
extensions: [],
showStudyList: true,
filterQueryParam: false,
servers: {
dicomWeb: [
{
    name: 'Orthanc',
    wadoUriRoot: 'http://localhost:8042/wado',
    qidoRoot: 'http://localhost:8042/dicom-web',
    wadoRoot: 'http://localhost:8042/dicom-web',
    qidoSupportsIncludeField: true,
    imageRendering: 'wadors',
    thumbnailRendering: 'wadors',
    enableStudyLazyLoad: true,
    supportsFuzzyMatching: true,
  },
],

},
// Extensions should be able to suggest default values for these?
// Or we can require that these be explicitly set
hotkeys: [
// ~ Global
{
commandName: 'incrementActiveViewport',
label: 'Next Viewport',
keys: ['right'],
},
{
commandName: 'decrementActiveViewport',
label: 'Previous Viewport',
keys: ['left'],
},
// Supported Keys: https://craig.is/killing/mice
// ~ Cornerstone Extension
{ commandName: 'rotateViewportCW', label: 'Rotate Right', keys: ['r'] },
{ commandName: 'rotateViewportCCW', label: 'Rotate Left', keys: ['l'] },
{ commandName: 'invertViewport', label: 'Invert', keys: ['i'] },
{
commandName: 'flipViewportVertical',
label: 'Flip Horizontally',
keys: ['h'],
},
{
commandName: 'flipViewportHorizontal',
label: 'Flip Vertically',
keys: ['v'],
},
{ commandName: 'scaleUpViewport', label: 'Zoom In', keys: ['+'] },
{ commandName: 'scaleDownViewport', label: 'Zoom Out', keys: ['-'] },
{ commandName: 'fitViewportToWindow', label: 'Zoom to Fit', keys: ['=']
},
{ commandName: 'resetViewport', label: 'Reset', keys: ['space'] },
// clearAnnotations
{ commandName: 'nextImage', label: 'Next Image', keys: ['down'] },
{ commandName: 'previousImage', label: 'Previous Image', keys: ['up'] },
// firstImage
// lastImage
{
commandName: 'previousViewportDisplaySet',
label: 'Previous Series',
keys: ['pagedown'],
},
{
commandName: 'nextViewportDisplaySet',
label: 'Next Series',
keys: ['pageup'],
},
// ~ Cornerstone Tools
{ commandName: 'setZoomTool', label: 'Zoom', keys: ['z'] },
// ~ Window level presets
{
commandName: 'windowLevelPreset1',
label: 'W/L Preset 1',
keys: ['1'],
},
{
commandName: 'windowLevelPreset2',
label: 'W/L Preset 2',
keys: ['2'],
},
{
commandName: 'windowLevelPreset3',
label: 'W/L Preset 3',
keys: ['3'],
},
{
commandName: 'windowLevelPreset4',
label: 'W/L Preset 4',
keys: ['4'],
},
{
commandName: 'windowLevelPreset5',
label: 'W/L Preset 5',
keys: ['5'],
},
{
commandName: 'windowLevelPreset6',
label: 'W/L Preset 6',
keys: ['6'],
},
{
commandName: 'windowLevelPreset7',
label: 'W/L Preset 7',
keys: ['7'],
},
{
commandName: 'windowLevelPreset8',
label: 'W/L Preset 8',
keys: ['8'],
},
{
commandName: 'windowLevelPreset9',
label: 'W/L Preset 9',
keys: ['9'],
},
],
cornerstoneExtensionConfig: {},
// Following property limits number of simultaneous series metadata
requests.
// For http/1.x-only servers, set this to 5 or less to improve
//  on first meaningful display in viewer
// If the server is particularly slow to respond to series metadata
//  requests as it extracts the metadata from raw files everytime,
//  try setting this to even lower value
// Leave it undefined for no limit, sutiable for HTTP/2 enabled servers
// maxConcurrentMetadataRequests: 5,
whiteLabeling: {
createLogoComponentFn: function(React) {
  return React.createElement('a', {
    target: '_self',
    rel: 'noopener noreferrer',
    className: 'header-brand',
    href: '/',
    style: {
      display: 'block',
      textIndent: '-9999px',
      background: 'url(../assets/log.jpg)',
      backgroundSize: 'contain',
      backgroundRepeat: 'no-repeat',
      width: '200px',
    },
  });
},

},
};
then i do:  yarn run dev:orthanc
But i have no changes, do you know what im doing wrong?
Thanks in advance
El sáb., 5 sept. 2020 a las 6:41, kavijoshi92<EMAIL_ADDRESS>escribió:

Change OHIF Logo on top left : Whitelabeling

Prior to build, add the following white labeling code in
Viewers>platform>viewer>public>config>default.js
Save your logo file in Viewers>platform>viewer>public>assets

window.config = {
.......
whiteLabeling: {
createLogoComponentFn: function(React) {
  return React.createElement('a', {
    target: '_self',
    rel: 'noopener noreferrer',
    className: 'header-brand',
    href: '/',
    style: {
      display: 'block',
      textIndent: '-9999px',
      background: 'url(../assets/Your_Logo.png)',
      backgroundSize: 'contain',
      backgroundRepeat: 'no-repeat',
      width: '200px',
    },
  });
},

},
....
}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OHIF/Viewers/issues/2029#issuecomment-687605633, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJXU667UAGN6AUKFGVJKHJ3SEIWWLANCNFSM4Q2THBWA
.

--
Atte.
Franklin Rodríguez
<EMAIL_ADDRESS>
[image: Please consider the environment before printing]

There are several ways to launch a dev server, based on method, the config files are chosen as mentioned in scripts inside package.json.
yarn run dev selects config file "default.js"
yarn run dev:orthanc selects config file "docker_nginx-orthanc.js".
if you are using "yarn run dev:orthanc" then put the whitelabel code inside "docker_nginx-orthanc.js".

Thank you so much!
It works, now i want to know something else, there is any way to apply
access control to OHIF working with ORTHANC? i mean like an user and
password authentication?
Thanks in advance
El sáb., 5 sept. 2020 a las 13:25, kavijoshi92<EMAIL_ADDRESS>escribió:

There are several ways to launch a dev server, based on method, the config
files are chosen as mentioned in scripts inside package.json.
yarn run dev selects config file "default.js"
yarn run dev:orthanc selects config file "docker_nginx-orthanc.js".
if you are using "yarn run dev:orthanc" then put the whitelabel code
inside "docker_nginx-orthanc.js".
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OHIF/Viewers/issues/2029#issuecomment-687651928, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJXU66ZTFBLFH4YGXOKG3C3SEKGAZANCNFSM4Q2THBWA
.

--
Atte.
Franklin Rodríguez
<EMAIL_ADDRESS>
[image: Please consider the environment before printing]

Please go through the OHIF documentation, https://docs.ohif.org/deployment/recipes/user-account-control.html.
User Account Control is another topic and should be asked in another issue.
If the whitelabeling is resolved you should close this issue.

Thankyou so much
What i have to do to close it?
El sáb., 5 sept. 2020 a las 20:39, kavijoshi92<EMAIL_ADDRESS>escribió:

Please go through the OHIF documentation,
https://docs.ohif.org/deployment/recipes/user-account-control.html
http://url.
User Account Control is another topic and should be asked in another issue.
If the whitelabeling is resolved you should close this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OHIF/Viewers/issues/2029#issuecomment-687690173, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJXU66Y2CNPHART4JKJRBWTSELY5ZANCNFSM4Q2THBWA
.

--
Atte.
Franklin Rodríguez
<EMAIL_ADDRESS>
[image: Please consider the environment before printing]

"Close issue" button adjacent to the "comment" button at the bottom. Only issue creator or project collaborator can close the issue.


Brew auto-changes branch when developing formula

Please follow the general troubleshooting steps first:

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?
[x] If you're seeing permission errors tried running sudo chown -R $(whoami) $(brew --prefix)?

Bug reports:
I was developing an update to formula in homebrew-core and when I tried to develop it, brew auto-switched back to master. I talked to @tdsmith and he indicated I should have set HOMEBREW_DEVELOPER=1 to bypass this new behaviour.
In short, this was surprising. I understand there was no way to warn about this, but it would have useful to know about. Perhaps we can put it homebrew-core's README/CONTRIBUTING/TROUBLESHOOTING documentation.

One other thing, it would be nicer if it poped the changes/ignored dirty taps, since then you are most likely working on something. Or just warn that it is dirty/not on master.

It does this if you have HOMEBREW_DEVELOPER set. I'll provide a decent response to this beyond that workaround in future. The short version: the hard thing here is getting the balance for Homebrew developers vs. those who don't know what Git is or really understand how to use it.

Have opened a PR for this in https://github.com/Homebrew/brew/pull/881.


Windows IME with Chrome(Edge) gets duplicated when typing symbols

Describe the bug
When using IME(Input Methods), input character right after some **Bold** part, the character gets duplicated.
To Reproduce
Steps to reproduce the behavior:

Just go to the website (https://ui.toast.com/tui-editor) and the Basic Editor part
Type something in **Bold**
Change to WYSIWYG mode, click the "B" button (this is to make sure the cursor is placed after the last *)
Use Emoji Keyboard (Press Win+.) and select one character, you can see the character gets duplicated.
(In my following scree recording I chose @ but actually you can choose any character, even an emoji; or you can use some IME such as Chinese Pinyin and type some symbols such as the full-width ，)

Expected behavior
Only one character should be inputted, instead of two characters.
Screenshots

Desktop (please complete the following information):

OS: Windows 10 1909
Browser: Chrome (or Edge, seems like a Chromium bug)
Version: 99

Additional context
I used the same scenario in Internet Explorer but it works fine in IE.
(Maybe you have to try for 2-3 times to successfully reproduce the bug.)

@AndyLiu9 Thank you for the issue.
We'll check it. As you know, IME characters are always causing problems in the browser's editable content area.


A hands-on approach to CSS Flexbox

Microbyte title: Hands-on approach to CSS Flexbox**
Domains covered:

HTML
CSS

Scope of learning for learners:

Learners will get to align HTML elements using CSS flexbox.
Learners will become familiar with implementing responsive design using flexbox.

Activities:

Working with nested <div> tags in HTML
Aligning HTML elements using CSS flexbox properties.

Make sure you follow the proper workflow. Don't make it too lengthy(max 1hr) and also don't make it tutorialish. Give room for contributors to contribute to your repo !

@sandip2224
For activities, you should add bullet points, like:
Activities:

Activity1
Activity2
Activity3
Follow the same for scope of learning for learners. Also add a short description, like a one-liner for each bullet point, don't just casually mention them.

Following this flow with aid in thinking clearly about your project. I hope you have some direction now, so kindly incorporate the changes requested in the issue description.
Last but not the least, kindly refer this issue Crio-Bytes/Miscellaneous#3 for reference.


Browserify conversion (fixed target)

This is the fix of #1014 to point to the dev branch, rather than master.

I am working on converting the build process to use browserify, so eventually better modularization can be achieved.
I've looked through the previous attempts at https://github.com/drkibitz/node-pixi and https://github.com/pixijs, but found they suffered from being too radically divergent. Unlike those, I'm avoiding making radical changes to the codebase. I hope to be able to track upstream with minimal conflicts, allowing gradual conversion to occur.
 But it allows the current code to work almost entirely unchanged while still splitting it up into unique modules in the browserify bundle. This should allow files to be converted to commonjs one at a time, rather than in one big rewrite.
I could use some feedback. Does this seem like a reasonable approach?
A possible next step to avoid the dependency in each file of a global "PIXI" variable being present would be to add var PIXI = require('path/to/module.js') at the top of each file. I've held off on that for now because it would break the existing build process, leaving browserify the only option. I want to get feedback before I continue.

Awesome work @Qard Pixi has been browserified in the rewrite branch, for more info check out #1296.

I think this switch to browserify isn't the greatest. Browserify was built for enabling server code running on the client side. Pixi is a pure client side library.
When using Pixi as a dependency for libraries, a granular control over the files you want included in the build process is no longer possible. You are limited to Pixi's controlled module bundles through --exclude
Here is an example of file daisy picking in 1.6.1
https://github.com/Boycce/pixi.js/blob/e4579b1515e7cc884c0b447ff791ae52e62f8c16/Gruntfile.js

@Boycce Still possible with browserify:
// loads only the Container.js file (and dependencies).
require('pixi.js/src/core/display/Container');

Cheers @englercj. Got it working in the end through browserify's b.pipeline.get('deps')...) & b.exclude('src') . Although not as convenient.

I agree, we are looking at doing ES6 modules in v4 which has the feature of being able to pull in only specific parts. Also, and extreme lean towards modularity to more granular control over what your bundle is.


Couldn't POST missing translations

Describe the bug

POST missing translation doesn't work
i18nextXHRBackend.js:90 POST http://domain/static/locales/en/common.missing.json 404 (Not Found)

Occurs in next-i18next version
0.53.0, 1.1.0
Steps to reproduce
const nextI18NextConfig: InitConfig = {
    defaultNS: "common",
    localePath: isBrowser ? "static/locales" : "src/static/locales",
    defaultLanguage: "en",
    otherLanguages: ["de"],
    keySeparator: "###",
    saveMissing: process.env.NODE_ENV !== "production",
    saveMissingTo: "all",
    use: [initReactI18next],
};

Expected behaviour
I expect missing translation are posted as a file.

I'm just guessing that there is a problem with structure, we have src folder for next sources, but CWD is placed one directory behind.

The problem was missingKeyHandler wasn't attached to server out of the box here.
@isaachinman Do you have any plans to use it for backward compatibility?
It already includes middleware package as dependency, I don't see a problem to attach it if config includes missing translation settings.

@ivansky I don't see a problem changing that. Should be a one-liner, and we can always save it for a v2 release if we're worried about breaking changes.

@ivansky I'm going to close this, as missingKeyHandler will probably require a custom implementation for each user. Thanks for tracking this down.


[android] fix gradle lint issues

Problem
There are lint errors when build the Android App with ./gradlew build.

Summary of Changes
This PR fixes the gradle lint issues:

Remove unused OnOffActivity and related resources;
Add Creator for classes that implements Parcelable interface;
Remove unnecessary call to ProgressBar.setMin/Max which requires API level of 26 (> current 24).

Fixes #3331.

Need to fix the style issue before pullapprove will added the rest of the reviewers.


zappa certify doesn't map domain name

Trying to follow the Using Let's Encrypt with Zappa guide. All steps work fine without any error message. However, accessing the custom domain name does not work. Looking into the AWS console I can see no changes to my Hosted Zone in Route 53 and nothing is added to my CloudFront. However, trying to run zappa certify once again results in an error:
Calling certify for stage dev..
Are you sure you want to certify? [y/n] y
Certifying domain test.smolinscy.se..
Setting DNS challenge..
Waiting for DNS to propagate..
Deleting DNS challenge..
An error occurred (BadRequestException) when calling the CreateDomainName operation: The domain name you provided is already associated with an existing CloudFront distribution.
Remove the domain name from the existing CloudFront distribution or use a different domain name.
If you own this domain name and are not using it on an existing CloudFront distribution, please contact support.
Failed to generate or install certificate! :(

That makes me curious. Where is that domain defined? I cannot find it anyware in my AWS console. Which existing CloudFront distribution does it refer to?
Steps to Reproduce

Follow Using Let's Encrypt with Zappa
Access the given custom domain url

Your Environment

Zappa version used: 0.42.1
Operating System and Python version: Ubuntu 17.04.
The output of pip freeze:

argcomplete==1.8.2
base58==0.2.4
boto3==1.4.4
botocore==1.5.40
certifi==2017.4.17
chardet==3.0.4
click==6.7
docutils==0.13.1
durationpy==0.4
Flask==0.12.2
future==0.16.0
futures==3.1.1
hjson==2.0.7
idna==2.5
itsdangerous==0.24
Jinja2==2.9.6
jmespath==0.9.3
kappa==0.6.0
lambda-packages==0.15.1
MarkupSafe==1.0
olefile==0.44
Pillow==4.1.1
pkg-resources==0.0.0
placebo==0.8.1
python-dateutil==2.6.0
python-slugify==1.2.4
PyYAML==3.12
requests==2.18.1
s3transfer==0.1.10
six==1.10.0
toml==0.9.2
tqdm==4.14.0
troposphere==1.9.4
Unidecode==0.4.20
urllib3==1.21.1
Werkzeug==0.12
wsgi-request-logger==0.4.6
zappa==0.42.1

Your zappa_settings.py:

{
    "dev": {
        "s3_bucket": "<SECRET>",
        "app_function": "my_app.app",
        "domain": "test.smolinscy.se",
        "route53_enabled": true,
        "lets_encrypt_key": "account.key"
    }
}

Do you have any CloudFront distributions setup in AWS? Do you have a CF distribution anywhere else (i.e. another AWS account) that could be pointing at that domain?
Take a look at this thread: https://github.com/Miserlou/Zappa/issues/334

Thank you for quick reply! As far as I know, I don't have any CloudFront distributions set up. Is that a pre-requirement?
The domain name is newly registered at another registrar (Loopia). All I have done is created a Hosted Zone on my account pointed out in ~/.aws/config and set the DNS servers accordingly at Loopia. whois smolinscy.se shows the DNS settings correctly.
I might have a few other AWS accounts, but no credentials set up for them in any configuration file, so I have no idea how it could possibly do anything with them. And when doing zappa update, I can verify the lamda funcion on my AWS account is updated.
I will take a look at your thread now.

Ah oki, as hinted in #334 they are defined in the Custom Domain Names of API Gateway. Yet no DNS records are created for the domain names. Should zappa update the Hosted Zone configuration? Can I debug what is happening when running the certify command?

Solved by the following workaround:

after runnng zappa certify for the first time, go to Custom Domain Names of API Gateway
copy the Distribution Domain Name (xxxxxxx.cloudfront.net).
go to Hosted Zoned of Route 53
click your apex domain name
click Create Record Set.
enter name of your subdomain (same as in zappa_settings.py), type: CNAME and paste into value the clipboard from step 2.
click Create and wait for the nameservers to update.

Done. :) I didn't invent this solution, but it involved quite a bit of googling and guessing. Now the list is written like I would have liked to have it too many hours ago.

@johey  Thanks so much for this! I already spent several hours going through API Gateway, Lamba, Route 53, ... looking for answers and found your post!

So is adding the Record Set in Route 53 essential to getting it working with your custom domain?  That would be my guess, but its odd that Zappa doesn't do that for us, since the docs lead us to believe it does.
I guess there is also this -- #762

It would indeed be easier if Zappa created the CName entry since the domain has to be on Route 53.

Duplicates #1141.
Fixing in my branch for #516 .

Hello, Thanks you @johey for your comment ! Made my day... however I had to create an A record instead of a CNAME record. Does anyone add to do the same ? Any why I had to do that ?

i had www.bla.com as domain in my zappa_settings. Route53 didnt allow me to create bla.com as a CName. But did allow me to create A record. @benjaminlong


Store suddenly throwing errors

I have a site which I built some time ago and it's been fine.  It's Concrete5 v<IP_ADDRESS> with Community Store v<IP_ADDRESS>
It was fine until yesterday when the product and product list pages were showing an error about missing columns in the CommunityStoreProducts table:
pPriceMaximum
pPriceMinimum
pPriceSuggestions
I added these and it got the pages live again.

That seems like the package was updated at some point code, but didn't trigger an upgrade correctly.
What I'd try is do a full backup, then manually increment the package number of community store by something trivial (in the package controller.php file), but enough to think it needs to upgrade. Then trigger that upgrade in the dashboard.  What it should do is re-check all the entities with database fields and create any missing columns.

I tried that and click update but unfortunately it's still an issue.


Release azure-cli-batch 2.0.4

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Started releasing azure-cli-batch 2.0.4...

Release of 'azure-cli-batch' with version '2.0.4' successful. View at https://pypi.python.org/pypi/azure-cli-batch/2.0.4.

GitHub release created. https://github.com/Azure/azure-cli/releases.

Codecov Report

Merging #3277 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3277   +/-   ##
=======================================
  Coverage   70.27%   70.27%
=======================================
  Files         391      391
  Lines       25084    25084
  Branches     3813     3813
=======================================
  Hits        17629    17629
  Misses       6344     6344
  Partials     1111     1111

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 fdbe1a0...02a0b67. Read the comment docs.


Initial Layout for Rocket.Chat Flutter SDK #1

Description
This PR implements the layout and CI for Rocket.Chat Flutter SDK project as proposed in issue #1 . The project structure, CI/CD setup, and the dummy implementation of tests are included. The main packages created are rocket_chat_embeddedchat_component and rocket_chat_api.
Packages:

rocket_chat_embeddedchat_component: Will contain the SDK with Flutter UI components for Rocket.Chat, allowing for easy embedding of the chat interface into other Flutter applications.
rocket_chat_api: Provides the Dart-friendly API/stub for applications requiring in-depth modification of chat behaviour or using the chat engine as a messaging fabric.

Both packages include a sample app which will use the respective libraries to demonstrate the functionality and usage.
CI/CD:

Added a GitHub Actions workflow for continuous integration, building, and testing.
Included build scripts and tests for both packages and the sample app.
The CI/CD will keep evolving as we move further.

Tests:

Integrated dummy test directory for each package and their corresponding sample apps. This directory will contain unit_tests, integration_tests and widget_tests as we move along.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Need also to make sure the workflows are legit by adding one of the monorepo management tools.
Seems Melos ,  mono_repo,  flutter_monorepo_build_tools or other.

@Sing-Li I've added new .gitignore rules and added Melos as a monorepo management tool.

closes #1


Make message more compact & limit it to 3000 characters

Archive-notify failed due to the 3001 character limit:
https://github.com/catalyst-cooperative/pudl-archiver/actions/runs/7265550885/job/19795893314
I made the output a bit less verbose, truncated the output, and added the fallback "text" field mentioned in the error message above.
Tested by running
pudl_archiver --datasets eia860m  --sandbox --only-years 2018 2020 --summary-file eia860m.json

And then
./scripts/make_slack_notification_message.py --summary-files *-summary.json | pbcopy

And then pasting that output into https://app.slack.com/block-kit-builder/ (removing the top-level text field) to verify that it looked fine.

You're thinking instead of some code-formatted JSON blob:
{
  "CREATED": [
    "foo.csv",
    "bar.csv"
  ],
  "UPDATED": ...
}

Having that actually formatted as a markdown list?


DEC-1340 Indexing

Altered indexing to correctly account for children.
This required updating the rsolr gem from 1.0 to 2.0, which had a bug with only having one child, which I fixed in a fork of the repo.
I'm open to suggestions on what we want the identifier to be for parents. Right now it's just a string relation is_parent: "true" or "false"

Coverage decreased (-0.1%) to 95.631% when pulling f774bfd2db73831a2deea45c7c8a0e4be6d63ff0 on childIndexing into 2d27055b43ae4121f0a8bf27b66f1549b08fc5dc on master.

Coverage decreased (-0.1%) to 95.631% when pulling 3b99333a7893356d05fd4a20b9a0e833d05afd72 on childIndexing into 2d27055b43ae4121f0a8bf27b66f1549b08fc5dc on master.

Coverage decreased (-0.1%) to 95.594% when pulling eeb9b41c31f649d3300207ca0b0df32d11fc6500 on childIndexing into 2d27055b43ae4121f0a8bf27b66f1549b08fc5dc on master.

Coverage decreased (-0.1%) to 95.593% when pulling d084ca0d36128b8e497ea4c8d3ec89a8a6afe11f on childIndexing into 2d27055b43ae4121f0a8bf27b66f1549b08fc5dc on master.


Add CLI

Is your feature request related to a problem? Please describe.
Add command line interface to pretrain, fine-tune, evaluate models similar to litgpt
Describe the solution you'd like
A clear and concise description of what you want to happen. Use https://jsonargparse.readthedocs.io/
Add feature to interact hte library though a command line interface
Additional context
Add any other context or screenshots about the feature request here.

I'd be happy to take this up. Could you guys define what the CLI should be able to do (e.g., core features you'd like to see)? @rheasukthanker @aaronkl

I will take of pre-training script


big commit, pretty much have messages working.  Nice work as always Joe!


GCC 7: dereferencing type-punned pointer will break strict-aliasing rules

After upgrading to GCC 7, when compiling at -O2 I now get a warning about a strict aliasing violation:
mapbox/variant.hpp:701:16: error: dereferencing type-punned pointer will break strict-aliasing rules [-Werror=strict-aliasing]
         return *reinterpret_cast<T*>(&data);
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~

libstdc++ implements the storage type through a variadic union and I think that might be the only solution for variant storage that guarantees no undefined behaviour. But then again, I can't see how one would obtain two references to the variant storage of different types without their code being broken anyway (the highest risk would probably be some code inside this library being miscompiled because of strict aliasing optimizations). So maybe it's enough to disable this warning (which is what I am currently doing myself)?
#pragma GCC diagnostic push
#pragma GCC diagnostic ignored "-Wstrict-aliasing"
[relevant code]
#pragma GCC diagnostic pop

Duplicate of https://github.com/mapbox/variant/issues/148

I don't know how I missed that one, sorry ^^°


Fixes to coverage and generate HTML report

Generate HTML coverage report (optional)
Fixes

Ensure coverage code runs first (we need to instrument & setup hooks before loading any of the other modules. I.e. src/client/**/*.ts files cannot be loaded until we have setup nyc.
Hence moved nyc to the very top so its obvious.
Also ensured we wait for report to finish (its async)

Codecov Report

Merging #4221 (b5fd5c1) into main (cfe01ca) will increase coverage by 4%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            main   #4221     +/-   ##
=======================================
+ Coverage     71%     75%     +4%
=======================================
  Files        384     389      +5
  Lines      25151   25333    +182
  Branches    3598    3625     +27
=======================================
+ Hits       17862   19141   +1279
+ Misses      5897    4709   -1188
- Partials    1392    1483     +91

Impacted Files
Coverage Δ

src/client/datascience/jupyter/jupyterServer.ts
76% <0%> (-6%)
:arrow_down:

src/client/datascience/jupyter/jupyterSession.ts
82% <0%> (-5%)
:arrow_down:

...t/common/variables/environmentVariablesProvider.ts
77% <0%> (-4%)
:arrow_down:

src/client/common/cancellation.ts
79% <0%> (-3%)
:arrow_down:

src/client/datascience/jupyter/jupyterExecution.ts
72% <0%> (-2%)
:arrow_down:

...lient/datascience/jupyter/jupyterSessionManager.ts
72% <0%> (-2%)
:arrow_down:

...datascience/jupyter/liveshare/hostJupyterServer.ts
84% <0%> (-1%)
:arrow_down:

.../datascience/interactive-common/interactiveBase.ts
71% <0%> (-1%)
:arrow_down:

src/client/datascience/baseJupyterSession.ts
74% <0%> (-1%)
:arrow_down:

src/client/datascience/liveshare/liveshare.ts
66% <0%> (ø)

... and 88 more