Should the Hash id generated by XML event and respective JSON/JSON-LD event be same?

@RalphTro @Echsecutor
Hello,
We came across a scenario where Hash-id generated for XML event and its converted JSON/JSON-LD event is different so we wanted to confirm if it needs to be the same or if this is expected behavior?
For example:
We tried to generate an event hash following a simple event in XML format:
<?xml version='1.0' encoding='UTF-8'?>
<epcis:Document xmlns:epcis="urn:epcglobal:epcis:xsd:2" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:cbvmda="urn:epcglobal:cbv:mda:" schemaVersion="1.2" creationDate="2019-10-21T14:59:02.099+02:00">
  <EPCISBody>
    <EventList>
      <ObjectEvent>
        <eventTime>2019-10-21T10:00:30Z</eventTime>
        <eventTimeZoneOffset>+01:00</eventTimeZoneOffset>
        <action>OBSERVE</action>
        <bizStep>urn:epcglobal:cbv:bizstep:shipping</bizStep>
        <disposition>urn:epcglobal:cbv:disp:in_transit</disposition>
      </ObjectEvent>
    </EventList>
  </EPCISBody>
</epcis:Document>

For the above event we received the following event hash id:
ni:///sha-256;508c04775b92e088ff199ced46aa3120791f8a907430541fa64ab30c49aa6d9e?ver=CBV2.0
We converted the same event to JSON:
{
  "@context": [
    "https://gs1.github.io/EPCIS/epcis-context.jsonld"
  ],
  "type": "EPCISDocument",
  "schemaVersion": "1.2",
  "creationDate": "2019-10-21T14:59:02.099+02:00",
  "epcisBody": {
    "eventList": [
      {
        "type": "ObjectEvent",
        "eventTime": "2019-10-21T10:00:30.000Z",
        "eventTimeZoneOffset": "+01:00",
        "action": "OBSERVE",
        "bizStep": "shipping",
        "disposition": "in_transit"
      }
    ]
  }
}

For the above event we received the following event hash id:
ni:///sha-256;8001402aeb8e39fb2d595ed3ac479d7cd462343acbd2eb7db9912f30ec7c6283?ver=CBV2.0
As we can see for the same event in XML and JSON/JSON-LD format we are receiving different hash ids so we would like to confirm if this is expected?
We believe this maybe be happening due to the bare string notation used in JSON/JSON-LD event format due to which we have differences in some of the fields such as bizStep and disposition.
CC:
@dakbhavesh

Dear @Aravinda93 ,
Nice to hear from you and thanks for making us aware of this. You are correct that one and the same EPCIS event must result in an identical hash value irrespective if it is formatted in XML or JSON/JSON-LD. We anyway still have to make a couple of further adjustments @Echsecutor  and I intend to tackle after EPCIS/CBV 2.0's ratification.
Kind regards and all the best,
@RalphTro

Dear @RalphTro,
Thanks a lot for the response and confirmation :)
Wish you a great day ahead :)
Kind regards,
Aravinda

@Aravinda93 your suspicion is correct, the bare string notation is not yet properly normalised. We will fix this in https://github.com/RalphTro/epcis-event-hash-generator/pull/66

Dear @Aravinda93,
From my POV, this can be closed from my POV; all tasks related to this issue have been addressed.
Would you agree?

Dear @RalphTro
Since you have provided the confirmation that XML and JSON event hash should be same and we will be testing it during the development to ensure everything is correct, let us close this issue to avoid any confusion with actual issues.

Gladly!


Cant start making plots

Hi all,
I am completely new to linux (deleted my windows to plot faster)
But now i cant start to make plots i can't find out what i am doiing wrong.
this is the error
Traceback (most recent call last):
  File "/usr/local/bin/plotman", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.8/dist-packages/plotman/plotman.py", line 143, in main
    wait_reason = manager.maybe_start_new_plot(cfg.directories, cfg.scheduling, cfg.plotting)
  File "/usr/local/lib/python3.8/dist-packages/plotman/manager.py", line 131, in maybe_start_new_plot
    stdout=open(logfile, 'w'),
FileNotFoundError: [Errno 2] No such file or directory: '/home/chia/chia/logs/2021-05-02-13:16:31.log'

This is my config
# Default/example plotman.yaml configuration file

# Options for display and rendering
user_interface:
        # Call out to the `stty` program to determine terminal size, instead of
        # relying on what is reported by the curses library.   In some cases,
        # the curses library fails to update on SIGWINCH signals.  If the
        # `plotman interactive` curses interface does not properly adjust when
        # you resize the terminal window, you can try setting this to True.
        use_stty_size: True

# Where to plot and log.
directories:
        # One directory in which to store all plot job logs (the STDOUT/
        # STDERR of all plot jobs).  In order to monitor progress, plotman
        # reads these logs on a regular basis, so using a fast drive is
        # recommended.
        log: /home/chia/chia/logs

        # One or more directories to use as tmp dirs for plotting.  The
        # scheduler will use all of them and distribute jobs among them.
        # It assumes that IO is independent for each one (i.e., that each
        # one is on a different physical device).
        #
        # If multiple directories share a common prefix, reports will
        # abbreviate and show just the uniquely identifying suffix.
        tmp:
                - /mnt/nvme1/00
                - /mnt/nvme2/01
                - /mnt/ssd1/02
                - /mnt/ssd2/03

        # Optional: Allows overriding some characteristics of certain tmp
        # directories. This contains a map of tmp directory names to
        # attributes. If a tmp directory and attribute is not listed here,
        # it uses the default attribute setting from the main configuration.
        #
        # Currently support override parameters:
        #     - tmpdir_max_jobs
        tmp_overrides:
                # In this example, /mnt/tmp/00 is larger than the other tmp
                # dirs and it can hold more plots than the default.
                "/mnt/nvme1/00":
                        tmpdir_max_jobs: 3
                "/mnt/ssd1/02":
                        tmpdir_max_jobs: 6
                "/mnt/ssd2/03":
                        tmpdir_max_jobs: 3
                "/mnt/nvme2/01":
                        tmpdir_max_jobs: 3

        # Optional: tmp2 directory.  If specified, will be passed to
        # chia plots create as -2.  Only one tmp2 directory is supported.
        # tmp2: /mnt/tmp/a

        # One or more directories; the scheduler will use all of them.
        # These again are presumed to be on independent physical devices,
        # so writes (plot jobs) and reads (archivals) can be scheduled
        # to minimize IO contention.
        dst:
                - /mnt/hdd/plot

        # Archival configuration.  Optional; if you do not wish to run the
        # archiving operation, comment this section out.
        #
        # Currently archival depends on an rsync daemon running on the remote
        # host, and that the module is configured to match the local path.
        # See code for details.
        archive:
                rsyncd_module: plots
                rsyncd_path: /plots
                rsyncd_bwlimit: 80000  # Bandwidth limit in KB/s
                rsyncd_host: myfarmer
                rsyncd_user: chia
                # Optional index.  If omitted or set to 0, plotman will archive
                # to the first archive dir with free space.  If specified,
                # plotman will skip forward up to 'index' drives (if they exist).
                # This can be useful to reduce io contention on a drive on the
                # archive host if you have multiple plotters (simultaneous io
                # can still happen at the time a drive fills up.)  E.g., if you
                # have four plotters, you could set this to 0, 1, 2, and 3, on
                # the 4 machines, or 0, 1, 0, 1.
                #   index: 0

# Plotting scheduling parameters
scheduling:
        # Run a job on a particular temp dir only if the number of existing jobs
        # before tmpdir_stagger_phase_major tmpdir_stagger_phase_minor
        # is less than tmpdir_stagger_phase_limit.
        # Phase major corresponds to the plot phase, phase minor corresponds to
        # the table or table pair in sequence, phase limit corresponds to
        # the number of plots allowed before [phase major, phase minor]
        tmpdir_stagger_phase_major: 2
        tmpdir_stagger_phase_minor: 1
        # Optional: default is 1
        tmpdir_stagger_phase_limit: 1

        # Don't run more than this many jobs at a time on a single temp dir.
        tmpdir_max_jobs: 3

        # Don't run more than this many jobs at a time in total.
        global_max_jobs: 12

        # Don't run any jobs (across all temp dirs) more often than this, in minutes.
        global_stagger_m: 30

        # How often the daemon wakes to consider starting a new plot job, in seconds.
        polling_time_s: 20

# Plotting parameters.  These are pass-through parameters to chia plots create.
# See documentation at
# https://github.com/Chia-Network/chia-blockchain/wiki/CLI-Commands-Reference#create
plotting:
        k: 32
        e: False             # Use -e plotting option
        n_threads: 2         # Threads per job
        n_buckets: 128       # Number of buckets to split data into
        job_buffer: 4608     # Per job memory
        # If specified, pass through to the -f and -p options.  See CLI reference.
        #   farmer_pk: ...
        #   pool_pk: ...

ths is the response from plotman dirs
ssh: Could not resolve hostname myfarmer: Temporary failure in name resolution
     tmp        ready   phases
/mnt/nvme1/00      OK
/mnt/nvme2/01      OK
 /mnt/ssd1/02      OK
 /mnt/ssd2/03      OK
     dst        plots   GBfree   inbnd phases   pri
/mnt/hdd/plot   51      4351                    101
archive dirs free space:

plotman.txt
man/plotman.yaml

I'm working on making plotman provide a better error, but check that the log directory listed in the error, and configured in the yaml, exists and is writable.
Also, don't install plotman as root or in general into the system Python.  apt should be run as root when installing.  Editing files in /etc should be done as root.  But definitely don't run pip as root, such as with sudo or otherwise.  There are many options, but the instructions suggest installing into the env create when you installed chia.
If you are archiving, you need to configure it with a name or address that the system can resolve.  myfarmer is not setup for this.
Best bet is probably to check in on the #plotman channel on keybase.  You can get interactive help there from many different people.

Thanks trying a complete reinstall off linux and all the other parts for the 3th time maybe now i get lucky.
Also maybe i am that stupid but i can't find the plotman team on keybase

There is a #plotman channel in the chia team.  Sorry.


Unexpected exception after HASS Core 10 update

The integration no longer loads after upgrading os core to version 10.
Here are the logs
This error originated from a custom integration.

Logger: homeassistant.loader
Source: custom_components/kia_uvo/KiaUvoApiEU.py:11
Integration: Kia Uvo / Hyundai Bluelink (documentation, issues)
First occurred: 19 April 2023 at 07:45:30 (1 occurrences)
Last logged: 19 April 2023 at 07:45:30

Unexpected exception importing component custom_components.kia_uvo
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/loader.py", line 760, in get_component
    cache[self.domain] = importlib.import_module(self.pkg_path)
  File "/usr/local/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/config/custom_components/kia_uvo/__init__.py", line 25, in <module>
    from .utils import DEFAULT_DISTANCE_UNIT_ARRAY, get_default_distance_unit
  File "/config/custom_components/kia_uvo/utils.py", line 13, in <module>
    from .KiaUvoApiEU import KiaUvoApiEU
  File "/config/custom_components/kia_uvo/KiaUvoApiEU.py", line 11, in <module>
    import push_receiver
  File "/usr/local/lib/python3.10/site-packages/push_receiver/__init__.py", line 1, in <module>
    from .push_receiver import *
  File "/usr/local/lib/python3.10/site-packages/push_receiver/push_receiver.py", line 11, in <module>
    from .mcs_pb2 import *
  File "/usr/local/lib/python3.10/site-packages/push_receiver/mcs_pb2.py", line 33, in <module>
    _descriptor.EnumValueDescriptor(
  File "/usr/local/lib/python3.10/site-packages/google/protobuf/descriptor.py", line 796, in __new__
    _message.Message._CheckCalledFromGeneratedFile()
TypeError: Descriptors cannot not be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:
 1. Downgrade the protobuf package to 3.20.x or lower.
 2. Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates

Reinstalling the custom component and logging in from HASS UI with the Pin code resolved the issue.


Any idea why ?

I'm confused. Any idea why recast did that ? (part of mesh went to very bottom).

This can happen if you have a very tall bounding box. You can try to increase the number of bits for span heights to 16 (the max value):
https://github.com/recastnavigation/recastnavigation/blob/c40188c796f089f89a42e0b939d934178dbcfc5c/Recast/Include/Recast.h#L266
Or you can see if you can use a shorter bounding box.

Right, I increased voxel per 1 world unit amount .... it was10 and it was bad shape, so I did 30. Could be that. Let me try to decrease to 15-20 voxels.

Btw, why you said 16 is max? Fields are
public:
unsigned int smin :
    RC_SPAN_HEIGHT_BITS; ///< The lower limit of the span. [Limit: < #smax]
unsigned int smax :
    RC_SPAN_HEIGHT_BITS; ///< The upper limit of the span. [Limit: <= #RC_SPAN_MAX_HEIGHT]
    unsigned int area : 6;                   ///< The area id assigned to the span.
    rcSpanPtr next;                            ///< The next span higher up in column.
};

So 13 was max, otherwise it goes over 32 bits. And can use 24 bits (not 16) as well, since next is int 64

Any way, thanks. Lowered  .walkableHeight = 20; (down from 30) and set 16 bits, mesh is back whole.

These coordinates are stored and saved as 16-bit ints in further stages of the build process and also in the final mesh. Normal meshing processes will have millions of rcSpan so increasing its size will also affect RAM usage by a lot.

Yes, RAM consuming is important, but can be saved just buying extra 16Gb for 30$ on build server. However, bit manipulations take many of time actually, bit fields are not recommended to be used at least on Intels long ago.

You are free to do those things. Other people have different use cases and cannot prebuild their meshes.


Allow sheetConfig overrides in Playbooks

In GitLab by @Asacolips on Feb 25, 2021, 08:34
Summary
Enter the feature request or bug description here.
Labels (do-not-change)
/cc @Asacolips

In GitLab by @Asacolips on Jun 23, 2021, 08:01
I think there's primarily two ways to get this feature working:

Complete Override. In this version, each playbook would have a sheetConfig field much like the system's sheetConfig setting, and if it's not empty/null, that's what actors of that playbook would use for their sheet. It would completely replace the normal sheet config for a given actor.
Diff Override. In this version, each playbook would have a sheetConfig field, but the format would be different. Each major section (attrLeft, attrTop, stats, moveTypes, equipmentTypes) would have an exclude property that could be used to remove attributes that are present in the base sheet and an include property where new fields are created and added.

Both methods have their pros and cons. Option 1 gives total control over playbook sheets, while option 2 is likely easier to maintain over a long period of time since you only have to define which attributes should be removed and which attributes should be added.


Conditional Access and -Credential problem

Hello guys,
I have problem with all PowerShell Azure / cloud cmdlets.
I have recently activated conditional access for my service accounts. If I connect the service account directly via Connect-AzureAD it works like a charm.
It looks like the -Credential parameter could not sent the valid compliant status / Azure AD hybrid joined status of a device.
$User =<EMAIL_ADDRESS>$string = ConvertTo-SecureString "MYPASSWORD" -AsPlainText -Force $Credential = New-Object -TypeName System.Management.Automation.PSCredential -ArgumentList $User, $string Connect-AzureAD -Credential $Credential

AADSTS53001: Device is not in required device state: domain_joined. Conditional Access policy requires a domain joined device, and
the device is not domain joined.

Another error from MicrosoftTeams cmdlet

Connect-MicrosoftTeams : One or more errors occurred.: AADSTS53001: Device is not in required device state: domain_joined. Conditional Access policy requires a domain joined device,
and the device is not domain joined.
Trace ID: 0ea6e100-ae37-4c6d-9cce-c8c343c73b00
Correlation ID: ff183e63-8085-4012-9f55-cad916dd0ad9
Timestamp: 2020-08-03 14:51:07Z
At line:1 char:1

Connect-MicrosoftTeams -Credential $Credential

  + CategoryInfo          : NotSpecified: (:) [Connect-MicrosoftTeams], AadAuthenticationFailedException
  + FullyQualifiedErrorId : Microsoft.Open.Teams.CommonLibrary.AadAuthenticationFailedException,Microsoft.Open.Teams.CommonLibrary.ConnectMicrosoftTeams

How can I workaround this problem?
Best regards
Robin

Hi @A141-Frech
Thank you for your issue submission.  Unfortunately, this appears to be a problem with the cmdlt and a support issue.  Issues in this repo are for documentation issues.  You can try one of the following links for additional help.
Microsoft Support:  https://support.microsoft.com/
Azure AD forums: https://social.technet.microsoft.com/Forums/en-US/home?forum=windowsazureaditpro
Stackoverflow: https://stackoverflow.com/
#please-close


Use YAML instead of INI for config file format

This is more fancy and extra parameters for the Media or the Instance objects can be passed as list, not simply as string.

Done in #31.


[localization Italian] per applicazioni di destinazione specifiche -> in modo mirato ad applicazioni specifiche

ℹ Computer information

PowerToys version: 0.25.0
PowerToy utility: Settings -> Keyboard Manager (Gestione Tastiera)
Language: Italian

📝 Provide where the issue is / 📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

❌ Actual phrase(s)
È anche possibile impostare i mapping per applicazioni di destinazione specifiche.
✔ Expected phrase(s)
È anche possibile impostare le mappature in modo mirato ad applicazioni specifiche.
ℹ Why is the current translation wrong
The English text is Additionally, mappings can be targeted to specific applications as well.
The current Italian translation contains two errors:

one error is the term mapping instead of using the Italian term mappatura as already reported in https://github.com/microsoft/PowerToys/issues/7852
the second error (that is the main topic of this issue) is the term di destinazione that is not appropriate in this context and makes the sentence awkward.

https://ceapex.visualstudio.com/CEINTL/_workitems/edit/332195

Great news!  This was resolved in the newly released 0.33 version of PowerToys!  Head to https://aka.ms/installpowertoys to grab latest.


recursivezero-internship assignment-color extractor

Please delete all the previous pull requests. I have made this fresh one.
The code includes working code, input/output examples, setup instructions, and assignment details in colorextractorREADME.md

Group A image have 6 images and we are resulting 4 colors only? similarly with image B.. we need exact colos atleast the count of images and there is no duplicate; color are diffrent for each stack of clock

Alright, I’ll look into it and do some changes. Also after doing the changes, should I fork the repo again and raise a new PR?
Get Outlook for iOShttps://aka.ms/o0ukef

From: Keshav Mohta @.>
Sent: Sunday, December 1, 2024 10:01:03 AM
To: recursivezero/assignment @.>
Cc: KAVISH GULATI - 122136039 - MITBLR @.>; Author @.>
Subject: Re: [recursivezero/assignment] recursivezero-internship assignment-color extractor (PR #19)
Group A image have 6 images and we are resulting 4 colors only? similarly with image B.. we need exact colos atleast the count of images and there is no duplicate; color are diffrent for each stack of clock
—
Reply to this email directly, view it on GitHubhttps://github.com/recursivezero/assignment/pull/19#issuecomment-2509566512, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BGP2HES4FNDR5MZTS7KLL7T2DKGIPAVCNFSM6AAAAABSYZWUQCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBZGU3DMNJRGI.
You are receiving this because you authored the thread.Message ID: @.***>

Disclaimer: This email and any files transmitted with it are privileged and confidential material of Manipal Academy of Higher Education (MAHE). They are intended to named recipient(s) only. If you are not the intended recipient of this message, please contact the sender and delete the message. This Email message including attachment(s), if any, is believed to be free of any virus. However, it is the responsibility of the recipient to ensure that it is virus-free, and MAHE accepts no liability for any loss or damage arising in any way from its use. Please consider the environment before printing this email.

I've updated the code by replacing colorextractor2.py with colorextractor3.py as per your feedback. Let me know if any further changes are needed.

I hard coded it as In the sample images I could see the colour count and like it wasn’t a user defined input code. If it would had been one, then I would have used a different approach.
Get Outlook for iOShttps://aka.ms/o0ukef

From: admin @.>
Sent: Tuesday, December 3, 2024 4:25:11 PM
To: recursivezero/assignment @.>
Cc: KAVISH GULATI - 122136039 - MITBLR @.>; Author @.>
Subject: Re: [recursivezero/assignment] recursivezero-internship assignment-color extractor (PR #19)
@recursivezero commented on this pull request.

In colorextractor3.pyhttps://github.com/recursivezero/assignment/pull/19#discussion_r1867499106:

       results["images"].append({

           "image_name": os.path.basename(image_file),

           "colors": colors

       })

   except Exception as e:

       results["images"].append({

           "image_name": os.path.basename(image_file),

           "error": str(e)

       })

return results

+if name == "main":

Sample images

image_files = ["GroupA.jpg", "GroupB-1.jpg"]
color_counts = [6, 9]

why we have hard coded color count? there might be 1 color or 30 colors at all
—
Reply to this email directly, view it on GitHubhttps://github.com/recursivezero/assignment/pull/19#pullrequestreview-2475300940, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BGP2HEVKUAYKUWYAZWCQOT32DWEY7AVCNFSM6AAAAABSYZWUQCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDINZVGMYDAOJUGA.
You are receiving this because you authored the thread.Message ID: @.***>

Disclaimer: This email and any files transmitted with it are privileged and confidential material of Manipal Academy of Higher Education (MAHE). They are intended to named recipient(s) only. If you are not the intended recipient of this message, please contact the sender and delete the message. This Email message including attachment(s), if any, is believed to be free of any virus. However, it is the responsibility of the recipient to ensure that it is virus-free, and MAHE accepts no liability for any loss or damage arising in any way from its use. Please consider the environment before printing this email.


Policy deleted when service instance may still exist

Description
Currently, the service broker will clean up the policy for the org and space when the service instance is deprovisioned, and the broker is not configured to preserve the policy. This currently does not account for the possibility of multiple Conjur service instances created in a given space.
How to Reproduce the Behavior
When a first service instance is created, the service broker will load the policy to define the policy and layer resources for the org and the space. When a second service instance is created, it will also load the same policy definitions, so there are now 2 service instances relying on the same Conjur resources for the org and space.
When either of these instance are deleted from the space, the service broker will remove the space resources (and the org, if there are no other spaces) without consideration for the other potential service instances relying on those Conjur resources.
Expected Behavior
Given a CloudFoundry instance when the Conjur Service Broker installed
And the "Preserve Policy" configuration is set to false
When I create a conjur service instance called conjur-1
Then the policy and layers exist for the org and space
When I create a conjur service instance called conjur-2
Then the policy and layers exist for the org and space
When I delete the conjur service instance called conjur-1
Then the policy and layers exist for the org and space
When I delete the conjur service instance called conjur-2
Then the policy and layers DO NOT exist for the org and space
Workarounds

Enable CONJUR_PRESERVE_POLICY so that the policy is never removed on deprovision
Only create a single Conjur Service Instance per space

@micahlee reading this again, I'm not sure we need to fix this yet - we could fix for now by adding instructions to our docs. I'll share this with PM for feedback.

also, we may address this by not allowing a second service instance to be provisioned (eg if on provision the space layer already exists, the provision should fail)

we may address this by not allowing a second service instance to be provisioned

Ah! I like that solution a lot. It was bothering me that we needed to come up with a solution for a situation that shouldn't really exist, and that would practically enforce the singleton nature of the service-instance per space.

it's nice to have some real things happening on provision now :)

Note: we need to take into account the "preserve policy" configuration

if preserve policy is off, then deprovision removes the space, and we can check for the existence of the space on provision to ensure there is only one service instance (and fail provision if the space exists already)
if preserve policy is on, then deprovision preserves the space - so we have no way of using the space layer to determine if the space already has a service instance. if someone deploys this way and later changes their config to "preserve policy" = off, this could be a mess.

is there a way to use the service instance ID in this logic @micahlee ?


Create sidharthviz.json

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

chore: profile addition

I will create new file..totally messed up sorry


Managing multiple slaves with the RTU API, perhaps concurrently.

For a use case I have I may need to talk to multiple slaves across a single RTU serial connection. It seems that the way to cater for this case is to update RTUClientHandler.SlaveId immediately prior to each IO so that the packets are always encoded with the correct slave id.
Is my reading of the code correct?
I could do this in my application code, but then I have to pass around the RTUClientHandler object as well as the Client which isn't something I was planning to do.
I might also want to handle concurrent access to the underlying handler from multiple goroutines. Again, I can add locks in my code to ensure this happens.
However, I am wondering if there is case for a per-slave proxy of the Client interface which transparently switches the slaveId on the underlying handler and acquires a shared lock prior to invoking the underlying Client. My application would then have one Client for each slave and they would all safely share the underlying RTUClientHandler.
I will probably add this kind of Client proxy to my own application, but if you think the idea would be useful to other users of your API let me know and I'll submit a PR. If I am fundamentally misunderstanding something about how the API is intended to be used, let me know that too!

wrt concurrent access, I'm in the process of adding mutex in the client transporter and fix #16 (expected to be done this week :) )
The client api doesn't have SlaveId in the parameter as using slaveId for TPC is rare. My idea originally is the ability to share the transporter layer among clients, which is like:
h := NewRTUClientHandler()
// Default client
c := NewClient(h)
// Create new Client for different SlaveId
h2 := NewRTUClientHandler()
h2.SlaveId = 2
c2 := NewClient2(h2, h)

May be I can add an helper
func NewClientWithSlaveId(c Client, slaveId int) Client {
   // Clone internal packager, set new slaveId and return a new Client with that packager
}

Please let me know if that will resolve your case. (Again, that requires #16 to fixed)

I think from my point of view, NewClientWithSlaveId(c Client, slaveId int) does what I need and has the advantage that if I need to defer the creation of clients for other slaves, that I can do so without carrying the handler around.

BTW: prototyping this, I realised that there is probably a need to add a Close() method to the Client interface and also a reference counter in the implementation to allow a Close() on the underlying handler to be deferred until there are no more references (otherwise this will have to be tracked separately).

I don't think Close is needed. Will send a PR for #16 this weekend

I had the same issue - which I solved by setting the Slave of the RTUClient before sending anything:
https://github.com/gonium/gosdm630/blob/master/src/github.com/gonium/gosdm630/modbus.go#L114
There is no need to Close() anything. I understand (and appreciate!) @nqv's view to provide a unified API independent of the transport, but for practical applications of the RTU transport, one needs to have a way to set the slave id.
HTH,
-Mathias


Is DeepCache by comfyui only supports sizes that are multiples of 32?

Describe the bug
When I was testing the acceleration inference of DeepCache, I encountered an error when I modified the inference size to 720*960. However, when I modified the inference size to a multiple of 8, there was no error.
This situation is similar to what I previously mentioned https://github.com/siliconflow/onediff/issues/527#issuecomment-1916449541 when testing the acceleration of diffusers.
Your environment
OS
Ubuntu 20.04.6 LTS

OneDiff git commit id
Building wheels for collected packages: onediff
  Building editable for onediff (pyproject.toml) ... done
  Created wheel for onediff: filename=onediff-0.13.0.dev1-0.editable-py3-none-any.whl size=5362 sha256=9e86fa1dc8cfefa40c1c163833bc6bf7e9c689012ae7b44dfab6e26ff4423d0f
  Stored in directory: /tmp/pip-ephem-wheel-cache-wqfwl901/wheels/b4/62/0d/0a502b53cf9cb2a2f46a88e2251c723c39df227d4f44413c4d
Successfully built onediff
Installing collected packages: onediff
  Attempting uninstall: onediff
    Found existing installation: onediff 0.13.0.dev1
    Uninstalling onediff-0.13.0.dev1:
      Successfully uninstalled onediff-0.13.0.dev1
Successfully installed onediff-0.13.0.dev1

OneFlow version info
libibverbs not available, ibv_fork_init skipped
Ignoring PCI device with non-16bit domain.
Pass --enable-32bits-pci-domain to configure to support such devices
(warning: it would break the library ABI, don't enable unless really needed).
path: ['/opt/conda/lib/python3.10/site-packages/oneflow']
version: 0.9.1.dev20240219+cu121
git_commit: 6621521
cmake_build_type: Release
rdma: True
mlir: True
enterprise: False

How To Reproduce
workflow_deepcache.json
The complete error message
Prompt executed in 1.54 seconds
got prompt
[ERROR](GRAPH:OneflowGraph_0:OneflowGraph) run got error: <class 'oneflow._oneflow_internal.exception.Exception'> Check failed: (45 == 46)
  File "oneflow/core/job/job_interpreter.cpp", line 307, in InterpretJob
    RunNormalOp(launch_context, launch_op, inputs)
  File "oneflow/core/job/job_interpreter.cpp", line 219, in RunNormalOp
    it.Apply(*op, inputs, &outputs, OpExprInterpContext(empty_attr_map, JUST(launch_op.device)))
  File "oneflow/core/framework/op_interpreter/eager_local_op_interpreter.cpp", line 84, in NaiveInterpret
    [&]() -> Maybe<const LocalTensorInferResult> { LocalTensorMetaInferArgs ... mut_local_tensor_infer_cache()->GetOrInfer(infer_args)); }()
  File "oneflow/core/framework/op_interpreter/eager_local_op_interpreter.cpp", line 87, in operator()
    user_op_expr.mut_local_tensor_infer_cache()->GetOrInfer(infer_args)
  File "oneflow/core/framework/local_tensor_infer_cache.cpp", line 210, in GetOrInfer
    Infer(*user_op_expr, infer_args)
  File "oneflow/core/framework/local_tensor_infer_cache.cpp", line 178, in Infer
    user_op_expr.InferPhysicalTensorDesc( infer_args.attrs ... ) -> TensorMeta* { return &output_mut_metas.at(i); })
  File "oneflow/core/framework/op_expr.cpp", line 602, in InferPhysicalTensorDesc
    physical_tensor_desc_infer_fn_(&infer_ctx)
  File "oneflow/user/ops/concat_op.cpp", line 55, in InferLogicalTensorDesc
    CHECK_EQ_OR_RETURN(in_desc.shape().At(i), out_dim_vec.at(i))
Error Type: oneflow.ErrorProto.check_failed_error
Exception in __call__: e=Exception('Check failed: (45 == 46) \n  File "oneflow/core/job/job_interpreter.cpp", line 307, in InterpretJob\n    RunNormalOp(launch_context, launch_op, inputs)\n  File "oneflow/core/job/job_interpreter.cpp", line 219, in RunNormalOp\n    it.Apply(*op, inputs, &outputs, OpExprInterpContext(empty_attr_map, JUST(launch_op.device)))\n  File "oneflow/core/framework/op_interpreter/eager_local_op_interpreter.cpp", line 84, in NaiveInterpret\n    [&]() -> Maybe<const LocalTensorInferResult> { LocalTensorMetaInferArgs ... mut_local_tensor_infer_cache()->GetOrInfer(infer_args)); }()\n  File "oneflow/core/framework/op_interpreter/eager_local_op_interpreter.cpp", line 87, in operator()\n    user_op_expr.mut_local_tensor_infer_cache()->GetOrInfer(infer_args)\n  File "oneflow/core/framework/local_tensor_infer_cache.cpp", line 210, in GetOrInfer\n    Infer(*user_op_expr, infer_args)\n  File "oneflow/core/framework/local_tensor_infer_cache.cpp", line 178, in Infer\n    user_op_expr.InferPhysicalTensorDesc( infer_args.attrs ... ) -> TensorMeta* { return &output_mut_metas.at(i); })\n  File "oneflow/core/framework/op_expr.cpp", line 602, in InferPhysicalTensorDesc\n    physical_tensor_desc_infer_fn_(&infer_ctx)\n  File "oneflow/user/ops/concat_op.cpp", line 55, in InferLogicalTensorDesc\n    CHECK_EQ_OR_RETURN(in_desc.shape().At(i), out_dim_vec.at(i))\nError Type: oneflow.ErrorProto.check_failed_error')
Recompile oneflow module ...
Building a graph for <class 'register_comfy.deep_cache_unet.DeepCacheUNet'> ...
Prompt executed in 41.97 seconds

We have fixed this, please update and have a try.

too old to follow, please feel free to reopen it.


Disable the EmsRefresh resume-on-error behaviour in specs

... except for the specs that are actually testing it, of course.
This way, when something goes wrong, instead of an unhelpful error like "unused HTTP interactions left in the cassette" or "expected: 10 got: 0", rspec directly reports the actual problem, complete with backtrace.

Interesting approach...@kbrock will like this one
cc @blomquisg


CLI - should verify that jsx is set

the CLI should verify tsconfig.json and give warning if compilerOptions.jsx is not set to react-jsx
 "compilerOptions": {
    "jsx": "react-jsx",
}

update cookbooks


Addon is forcefully tracking the weekly world boss

Fixed by un-tracking it if it wasn't tracked before the check

Fixed with a5faafd


Deleted elements preventing persistence

Hi all, we've forked the Excalidraw client over at https://github.com/peoplerenaissance/excalidraw and have been loving it so far.
Changes in our fork include swapping the backend from Firebase to an HTTP REST API that supports the same persistence operations (among other UI adjustments for our product's use case), and we've had many successful collaborative drawing sessions so far.
Unfortunately, today we discovered a drawing that won't automatically save, which I believe is related to https://github.com/excalidraw/excalidraw/pull/5108, filtering deleted elements prior to persisting:

For our use case, I'll make a temporary fix in our fork to sync all elements, including deleted ones, but I'd love some pointers into understanding how this might be caused and if we can prevent it in the future, or if we can help contribute to an upstream fix.
Thanks!

@ummcheng I opened #5108 because over the course of a semester, the boards I taught on stopped saving to Firebase due to deleted elements filling up storage.  It sounds like you're seeing similar symptoms, if for different reasons.  CC @dwelle on the recurrence of scenes failing to save.

@ummcheng could you share little more context on why filtering out deleted elements is an issue ? We don't save the elements which are deleted and is not updated in last 24 hours.  Are you seeing some inconsistency due to this ?

Appreciate the prompt attention and help from both of you.
Yes, it seems like somehow the drawing we have saved in storage has deleted elements that have not been updated in the last 24 hours. On the initial load, all of the elements are included in the calculated scene version, which in this case is 7977. Then when other changes are made, the scene version calculation excludes those deleted elements (in this case resulting in 6643, and doesn't attempt to the save the scene, even though the scene is now "newer".
I haven't fully traced the lifecycle of deleted elements and how they're updated but I wonder if this is the simplified version of what we experienced:

A scene is created (no elements, so calculated version is 0).
An element is created and added to the scene (calculated version is 1). The scene is saved.
The element is deleted and the scene is still saved to storage since the update time is < 24 hours (version 2).
24 hours later, the scene is loaded and the client calculates the scene version is 2 (including the deleted element).
When the save handler is called, the scene version is calculated to be 0 since the deleted element is filtered out, and since the version stored is 2, the save is skipped.

Does that sound plausible at all?

I haven't fully traced the lifecycle of deleted elements and how they're updated but I wonder if this is the simplified version of what we experienced:

A scene is created (no elements, so calculated version is 0).
An element is created and added to the scene (calculated version is 1). The scene is saved.
The element is deleted and the scene is still saved to storage since the update time is < 24 hours (version 2).
24 hours later, the scene is loaded and the client calculates the scene version is 2 (including the deleted element).
When the save handler is called, the scene version is calculated to be 0 since the deleted element is filtered out, and since the version stored is 2, the save is skipped.

Does that sound plausible at all?

It does sound plausible, and is most like what you're seeing in your project.
This specific problem shouldn't affect excalidraw.com, because we're filtering out the "dead" elements on load before we cache the version:
https://github.com/excalidraw/excalidraw/blob/3d5356cb8edc043a721a5e66167b2814b191ebc2/src/excalidraw-app/data/firebase.ts#L290-L296
Whereas in your project you're not doing so:
https://github.com/peoplerenaissance/excalidraw/blob/746b55362bf620a5e7041399f64169d0500be94a/src/excalidraw-app/data/httpStorage.ts#L112-L117

Oh wow, thanks so much for digging in our fork and discovering that! We did remove the encryption functionality and I must have totally missed using getSyncableElements() on the initial load.
Thanks again for the help even though this doesn't affect upstream Excalidraw! Will close this issue 🎉


Add Sitemap

Add Google Sitemap for helping with indexing of the blog

Why this? I added the sitemap to website with Articulate blog. http://www.volaresereni.it/sitemap

Yeah, it can be easily added manually but it would be good to add it to the core.
It would be cool if you could send a PR :)
Sent from my iPhone

On 11 Jan 2017, at 22:05, BP<EMAIL_ADDRESS>wrote:
Why this? I added the sitemap to website with Articulate blog. http://www.volaresereni.it/sitemap
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes. I used the good package CM Umbraco Tools where you can exclude some type of pages from sitemap https://our.umbraco.org/projects/developer-tools/cm-umbraco-tools/ .
The only problem that I see with Articulate is that if you link to authors, I return the page not found. Others main issue is we don't have the tags and categories into the sitemap because they are not a really pages. So I think that we need to specialize the CM Umbraco Tools ( good think ) or build a package for Articulate.

Indeed, a custom site map would be needed inside articulate to take care of tags/categories
Sent from my iPhone

On 12 Jan 2017, at 07:50, BP<EMAIL_ADDRESS>wrote:
Yes. I used the good package CM Umbraco Tools where you can exclude some type of pages from sitemap https://our.umbraco.org/projects/developer-tools/cm-umbraco-tools/ .
The only problem that I see with Articulate is that if you link to authors, I return the page not found. Others main issue is we don't have the tags and categories into the sitemap because they are not a really pages. So I think that we need to specialize the CM Umbraco Tools ( good think ) or build a package for Articulate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Specs?

Just want to chime in and hope that a friendly soul will help out with this feature :)


Incorrect cyrillic text for some feeds

Some feeds shows with abracadabra:
Examples for http://filmitorrent.org/rss.xml:

Same feed item in Feedly shows correctly:

https://github.com/mjibson/goread/issues/346


command to retrieve the IP addresses from the DNS and from the peer to peer network

ariadne network discover

depends on #95

uses https://doc.rust-lang.org/std/net/trait.ToSocketAddrs.html to get the list of IP addresses associated to a given peer


⚠ Bofrost.es has degraded performance

In 211c6d7, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1460 ms

Resolved: Bofrost.es performance has improved in bc3d480 after 6 minutes.


fix initial render

Fixes initial render. Meant to have this be fixed weeks ago — but forgot. Going to merge once tests pass (:

v1.0.2


Remove unnecessary explicit dependencies: sprockets, activesupport, sass and pry

With the exception of pry, none of these dependencies are explicitly referenced in the code (activesupport and sprockets are implicit though). This removes an unnecessary constraint on sprockets 4, as middleman-sprockets depends on 3 and up which causes problems for apps that integrate with govuk_publishing_components.
⚠ Don't forget to update the gem version in the CHANGELOG before merging! When you're ready to release bump version file and generate a tag. ⚠

Do you want to bump the version to 2.2.1 in this PR, so we get a new gem when we merge?

Do you want to bump the version to 2.2.1 in this PR, so we get a new gem when we merge?

I'm optimistically hoping I might get my long protracted bug fix PR into this release too. But if that hits a snag I'll release them separately.


Display of a columnedList view fails

In v1.0.1139, I get the following assertion failure when rendering a columnedList view:

The fix is inserting asString after eachColumn title.

What is the source code of the view? Normally the title of the view should be set to a string.

The title of the view is a string, but not the title of the column, which is a BrRopedText. I have lots of such views, and they have been working fine for a long time.
Here's the source of the specific view I was opening, but it probably doesn't make sense out of context:
gtItemsFor: aView
	<gtView>
	| view |
	view := aView columnedList
		title: 'Items';
		priority: 1;
		items: [
			(1 to: sequence numberOfRows) collect: [ :row |
				{ row. sequence atRow: row } ] ].
	indexVariable
		ifNil: [
			view
				column: 'Index'
					text: [ :item |
						item first asRopedText foreground: Color gray  ]
					width: 50 ]
		ifNotNil: [
			view
				column: indexVariable
					text: [ :item |
						(item first * stepSize) asRopedText foreground: Color gray  ]
					width: 50 ].
	variables withIndexDo: [ :var :i |
		view
			column: var asLeibnizSyntax
			text: [ :item | item second at: i ] ].
	^ view

Aha. Missed that it was the title of the column. I think we assume in most places that the column title is also a string. We can add that asString call there, but it might be also safer to set column title to a string when creating the view. I see that change was made around a year ago.


Plugin 4.26  lots of errors compiling

I tried to use the plugin with the 4.26 engine. I am getting lots of errors while compiling. First lots of Duplicate leaf header names. Has anyone used the plugin without errors until now?

This is the only compile error that I'm getting:
Source/StreetMapRuntime/StreetMapSceneProxy.h(10): Error: Duplicate struct name: StreetMapVertex

I tried to use the plugin with the 4.26 engine. I am getting lots of errors while compiling. First lots of Duplicate leaf header names. Has anyone used the plugin without errors until now?

This is the only error that I'm getting:
Source/StreetMapRuntime/StreetMapSceneProxy.h(10): Error: Duplicate struct name: StreetMapVertex

Are you compiling with VS2019 or VS2017?

ya same issue here too.. its not working then ?

Any fix on this? Same problem with 4.27 and VS2022

the code is Duplicate.So i removed one of them.And i adjust some *.h file including order,then i got it runnable.

What adjustments did you do to the *.h including orders?
I'm getting errors regarding the "UnrealEd.h" include


[CMake] Tweak path separators in new Swift runpaths

@rintaro and @bnbarham, this is a quick tweak pull to fix the double slashes in the runpaths shown here after #68082:
> readelf -d usr/lib/swift/host/plugins/lib*so | ag "File:|runpath"
File: usr/lib/swift/host/plugins/libObservationMacros.so
 0x000000000000001d (RUNPATH)            Library runpath: [$ORIGIN/../../..//swift/linux:$ORIGIN/..]
File: usr/lib/swift/host/plugins/libSwiftMacros.so
 0x000000000000001d (RUNPATH)            Library runpath: [$ORIGIN/../../..//swift/linux:$ORIGIN/..]

@swift-ci Please smoke test

this doesn't affect the actual .so path resolution, right?

Yes, this should have no functional effect, purely for looks.

Linux CI broke because of my lldb rpath pull that was just merged, I'll update the failing integration test in a couple minutes.

OK, hopefully linux CI will pass now.

@swift-ci please smoke test Linux platform


Device tracker show devices as always Home

I have a problem with the device tracker. After a day or two some devices starts to show as Home, even if they haven't been for several hours. This is not a configuration problem, as it works flawless until the state gets stuck at "Home". I have encountered this problem with snmp, but i see on the forums that this is the same for nmap and ping also.
The fix seems to be to delete know_devices.yaml, and let hass populate know_devices again. For some this will work, and for some og doesn't. For me it works a couple of days after i delete known_devices.
The forum posts go back to 2016, so this is a bug that have been for a while.
See the forums similar cases: https://community.home-assistant.io/search?q=Always home

I've experienced something similar after stopping HA with some devices home and starting it up again when they are not. It seems previous "Home" devices will not be updated until they are seen again, which will trigger the stale check based on the consider_home config parameter. (I might be wrong though, I only quickly glanced at the device tracker code)
Not sure if this applies for you, but it might be the cause for some of the other similar cases mentioned in the forum.

That's interesting. It could very well be that I'm experiencing the same. I will have to do some tests and see if that could be it.
Either way, there's a bug somewhere..

Hi, I'm experiencing the same issue.
I also tried to modify the "consider_home" value because it seems to me being linked to "interval_seconds" value.
I feel the problems began after that modification. I deleted known_devices.yaml and will see what happens.
But nevertheless I agree it seems to be a bug. Might be the "consider_home" event not triggered after specified value is edited...

Hi i have the same problem to. Started today after update to 0.60.0.  On of two asuswrt devices is always on. Tried to restart both home assistant and router without any luck. Shouldn't Hass check asuswrt on startup for the current state?


Lens' generated private cannot be used to run Wavelet

Referring to issue #245.
Lens generate private key that has prefixed zero bits of 0.
This will cause issue if you use the key to run Wavelet, since the Wavelet's SKademlia requires the private key to have 1 prefixed zero bits (C1).

The possible solutions are:

Update Lens and/or Wavelet to make C1 to match.
Note: If we make Lens to fulfil the C1, it'll increase CPU usage of the Lens server.

Make C1 configurable, and have the user pass the correct value of C1.

Let's redirect this issue over to Lens rather in Wavelet.


Add isNonEmpty to Traversable?

Vavr has already Traversable.nonEmpty() which is aligned to Scala's Traversable.nonEmpty.


10 final lesiaturuta

Update I've added a link to your stage https://lesiaturuta.github.io/fe-course-basic-2023/

Hi @lesiaturuta! I'll check your project step by step

please add some paddings to the main block of the Login page

You can use the mockup as an example

It's just a piece of advice: try to avoid consolidating all styles into one file. Instead, it's better to create separate files for each page. This approach helps in navigating styles more effectively, preventing the addition of redundant styles across all pages. If you have one file with all styles, it can become excessively large and negatively impact page speed

You can resize font-sizes of the text and gaps to improve the mobile view:
Your current mobile view

Just example from me

Please be attentive to the fixed heights present in the style.css file. Kindly review the entire file and fix any instances where fixed heights are specified. It's recommended to ensure flexibility in height to accommodate varying content and screen sizes. Thank you!


Q: Why is the dummy agent created with a random suffix?

I noticed that for some pool instead of a "dummy-agent", an additional suffix is added.
E.g. dummy-agent-fa7973d57ce3f26c

thanks for your PR


Feat  save anon link #2283

Implements the anonymous-warning slide-in (based on issue #2283)
Fixes #2283
Anon slide-in is currently shown everytime a user is logged in and there is a privateId set in the url -> this should/needs to change once we return isAnonymousAccount after signup/transfer/login, so we can remove the privateId in url handling after all and just use it as a login entry point

Copy link to clipboard doesnt seem to work because of the z-index impl, we might need to use a different approach


Support different bigint implementations

PR changes BigInt - it was a type alias, became a structure wrapping gmp/num-bigint. BigInt hides underlying implementation - you can only manipulate it by calling traits implemented on BigInt (set of implemented traits remains the same regardless of chosen bigint implementation).
Exposed API is limited, but should be enough for most of the cases. It's also extensively tested that both implementations behave the same in public methods.
This PR affects curv API, so it requires major version bump (up to v0.6.0) and some migration work in upstream crates.

@omershlo, thank you for reviewing this big list of changes! I've just pushed fixes


Add ability to depend on latest PureScript release candidate via "unstable" version

Since setup-purescript is used across core, contrib, web, and node, we'll need a way for CI to use the latest release candidate when migrating to v0.15.0.
Changing the version to 'unstable' for PureScript should select the latest release candidate, so we don't have to update it again if a new release candidate is made.

#25 is related as well


(Julia) Fix chocolateyinstall.ps1 binary shimming

Description
This PR changes the way of searching for install location in chocolateyinstall.ps1 from Get-AppInstallLocation to directly Get-UninstallRegistryKey matching the current version being installed.
Motivation and Context
Previously, updating Julia when older versions are installed is problematic with shimming, because the multiple existing registry keys for Julia abort or mess up the install location search. This PR fixes Issue #1304.

How Has this Been Tested?
Installing/upgrading/uninstalling and script auto updating have been tested on my local Windows 10 machine and the test environment.

Screenshot (if appropriate, usually isn't needed):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[x] All files are up to date with the latest Contributing Guidelines
[x] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review

Wait... AppVeyor tests says

WARNING: Can't find Julia install location
This does not happen on my side. I do not know why and I might need to look into it.

:white_check_mark: Package verification completed without issues. PR is now pending human review

Thank you.


Added telegram_bot_bundle project (Symfony wrapper) to the list of projects using this library in the README.

?
!

Type
improvement

BC Break
no

Fixed issues

Summary
Added telegram_bot_bundle project (Symfony wrapper) to the list of projects using this library in the README.

Thanks @m4n50n :+1:


str.strip() doesn't do what they thought it did

The code has many instances of x.lstrip('prop ') for example. They though it would remove the keyword from the front, and it does. It also removes all the letters 'o', 'p', 'r', and ' ' from the left edge of the string. So, for example, 'prop roof'.lsplit('prop ') becomes 'f'
Will need to pick a better method everywhere this is used.

Fixed by 7ed90117dd843883f025cb8ff9405740d3e950df


No way to set the initial value of an Input if you use formField

This pull request fixes #5549

LGTM
Thanks!
(Ideally patches like this would come with a unit test showing the improvement so we don't mess it up in the future.)

Also at some point we should document it.
Thanks for the patch, @eukreign!


Fix the secret regexp pattern on web page

Before

After

However, the error is caused by the web page not handling the post form failure, which is 400 bad requests in this case. But I'm not sure where to add the error handle. Any advice is appreciated.

Should we add a config option for this pattern? I think it is used in many places in the backend as well so one shared definition will help for these patterns to no longer go out of sync. The regexp must be compatible with both the JS and Golang flavor, but that's easily verified on https://regex101.com/.

I was unable to create a backport for 1.20. @lng2020, please send one manually. :tea:
go run ./contrib/backport 26910
...  // fix git conflicts if any
go run ./contrib/backport --continue

The code doesn't exist in v1.20, so we don't need a backport


fix(TextArea): add basic styles and inverted variant

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS


Use cloneDeep to make deep copies of filters

Related issues

Short description and why it's useful

Some search filters contain arrays, so they need to be deep copied, to avoid polluting the config object. Previously filters were cloned with Object.assign(), but this results in arrays being copied by reference. Now using lodash's cloneDeep.
Screenshots of visual changes before/after (if there are any)

Which environment this relates to
Check your case. In case of any doubts please read about Release Cycle

[ ] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[x] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[ ] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[ ] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and currently important rules acceptance

[x] I read and followed contribution rules

Yeah, I understand. The issue comes from passing around the Config object, assigning parts of it to other variables and then passing those vars to other functions. It becomes very difficult to determine where the mutations are happening.
Of course the Config object should be absolutely read-only. When it's shallow copied, using Object.assign() or destructuring, then there's always a chance of causing problems. So, wouldn't it be safer to always clone the relevant section? Especially in cases like this, when we can't know what the contents are.
Anyway, if it's a concern, I can look into each one in more detail.
I'll comment on your suggestions. At least one of them needs multiple clone operations.

Of course the Config object should be absolutely read-only. When it's shallow copied, using Object.assign() or destructuring, then there's always a chance of causing problems. So, wouldn't it be safer to always clone the relevant section?

Not really, we can use config directly but we should always keep it mind that we can't do anything that can mutate it. For example when you picking just one section from the config, please don't operate on it later, create a new object instead (eg. with using spread operator) etc.
 const newObj = { ...config.section, yourModidication: ... }

instead of:
 const newObj = config.section
 newObj.yourModidication = ....

Of course mentioned config.section should be a shallow object.
I think you totally know what I mean.
@grimasod please update the changelog as well.


Thoughts on status/roadmap?

Hi,
Thanks for making this very nice library. I am considering using it in a project, but notice that it hasn't seen commits/PR merges in a year or so. So I wondered if you could share some thoughts about whether or not you consider it feature-complete, maintenance status, roadmap etc.

I'm afraid author is MIA.. but package is very nice and extendable. I'm using it in production, as are many others clearly.. there isn't really anything better out there that I've found.

Agreed. Seems like he's abondoned this project for some time now. I too use this project in production and have resorted to self-hosting a forked version to patch issues we were experiencing. Specifically related to PR #426 and issues #423 and #425

Hi guys, @rubjo, @vesper8, @Spacarar,
sorry I'm rather MIA on this library atm, but I'd really love to get things rolling for what is missing and/or not working as expected.
The thing is, it really takes a lot of time to dive in each PRs and issues and I am busy on other projects.
It frequently happens that I look into a PR and didn't merge because only a small portion of the problem is addressed and I don't have time to redo everything.
But I really appreciate your feedbacks all, and I am not dead yet. 🤘
So if someone wants to contribute in Vue Cal on a regular basis, I am happy to pair for some things, explain the codebase and where I left it, and discuss the next steps together. 🚀
If you're interested and and the codebase makes sense to you and doesn't scare you, reach out to me via email that you can find in the package.json. :)
If not, for now my priority is Wave UI.
❤

Big thanks to everyone for clarifying. I’m going ahead and will try this out for a small project - and will try to see if I can convince the higher powers that some financial support is in place for a useful open-source library like this if it is successful.
I haven’t actually come across any major bugs like the ones you refer to yet @Spacarar, I’m praying these won’t affect my use case. :)
I haven’t really looked much at the source codebase yet @antoniandre, so contributing to the code would maybe be out of my depths. If you find it helpful, though, I could probably go through and reply to some of the issues here - by the looks of them, several lack minimal reproductions of reported problems (and others seem very much more at home on Stack Overflow than on an issue tracker).

Thanks for your support.
Replying a few issues would definitely help, as well as classifying them with labels, and having financial support for more time investment.
I've just updated a few things - dependencies, vulnerabilities, and merged a couple of straightforward PRs.
the rest requires more time.
And for the cheering part, I will most likely work on this codebase again this year.

I've gone ahead and commented and asked for repro / more info on a lot of the issues.
Other issues are clearly inactive / missing responses after you have asked for more info, and should just be closed. I wasn't able to close off outdated ones or start organising the issues with labels.
TBH I would just close most of the ones not providing repros or not having seen any activity for a year or two.
And with Vue 3 now being the default, I think I would actually consider only focusing on the next codebase moving forward.

I'm maintaining a dozen projects built on Vue 2 with no intent, desire, or reason really, to upgrade to Vue 3. Vue 3 may be the default for new projects (which is arguable, but not looking to argue it here) but Vue 2 remains by far the most deployed and maintained version out in the wild. It's entirely up to you whether you want to only focus on the next branch but I think that would be unfortunate and would likely lead to other contributors backporting your changes into the Vue 2 branch.
Overall I've found your package to be quite stable and mostly feature-complete. The one feature request that would be awesome to have in the future, which seems to be quite involving, is the horizontal timeline: https://github.com/antoniandre/vue-cal/issues/87
I'm happy to hear you may find some time to work on this in the coming year, and thank you again for your contribution.
I also don't think this issue should have been closed : ) Perhaps it should be pinned instead.

I feel your reaction @vesper8 :) It was a suggestion on how to move forward to an author who has made something great, but has limited time, like anyone else. Maybe leaving other contributors backporting the changes into the Vue 2 branch could actually free up time for @antoniandre to make one version more stable, I don't know.
I agree that #87 sounds like a mammoth task, one that transforms the view completely and is perhaps better served by other libraries (?)
I can always reopen this for a while, but a roadmap or similar is probably best organised by the author himself and in a different way than a long-winded issue.

Hey guys!
Thanks for the help @rubjo, I'll close a few issues shortly, at least all the no-repro.
I understand your point @vesper8, and when I add more features/bug fixes, I will also add them in the vue 2 branch.
In a near future though, vue-cal will also default to the next branch to follow vue 3 as vue-cal depends on it.
But practically for users, it will only change how you install from npm.
When I'm back in the codebase, I will focus on smaller features first and refactoring a few things.
Meantime, anybody if you want to contribute, it's very appreciated. You can submit well-tested PRs that take all the cases in consideration, and I'll merge in if it looks complete. :)
@rubjo, if you wish to help things moving on, you can have a stab at #462 which is probably very easy to fix. :)

Closing this issue as I have already rebooted the project.
I don't have full time, but I will do a few improvements and bug fixes.


Why does edge (vertices are the most similar nodes of each other) have greater weight when compare with min_sim

thanks for your work!!! I love it very much!!
I want to know Why does edge (vertices are the most similar nodes of each other) have greater weight when compare with min_sim?
In the src code, the weight of edge(vertices are the most similar nodes of each other) is 2, while others is 1 when compared with min_sim

@seanM29 the weights of edges would be greater when they are directed and form loops, its just an output of the way clustering equation is implemented, however important to know is that FICNH does not use any edge weights, they may as well be all set to 1's (I leave this out to save some compute.. as it doesn't matter we just use all connected components irrespective of their weights). The comparison of min_sim is only optional (see the updated read me), it helps early exit and may  prove useful when your data is unbalanced. This however also can not be done when you are using approx NN (KD-tree)  to find the 1st nbr, only would run when using exact distance to compute 1st nbrs.


a bug 'Field 'result_info' doesn't have a default value'

A bug with laravel 5.3 ,that isField 'result_info' doesn't have a default value , fix it with $table->text('result_info')->nullable();

Thx!

已更新2.5.2

已更新2.5.2


Fix link to blog post

Was 404ing before.

Thanks for spotting this. Actually, it was the redirect from our old website that broke, so I fixed that too like the good web citizen I am! But this PR updates the link too, so merging it!


docs: pile of tweaks and additions

I was stuck on the train this afternoon with barely a working internet connection, so what better thing to do than close a whole pile of docs issues 😄
I tried to land everything (mostly) as separate commits. Changes included:

Change page titles and navigation links from Title Case to Sentence case
Remove ::pyo3 from guide samples (given that's not quite what the macro expands to anyway!)
Closes #2547
Closes #2546
Closes #2495
Closes #2122
Closes #1602
Closes #804

All reviews welcome, I'll probably merge this on Tuesday if I haven't heard anything so that it can be part of the published 0.17 guide. Figure even if it's not perfect it's probably better than not including in the release :)

Rendered


3.67 doesn’t work

As I don't wish to update my devices over 3.65, I usaly rely on nowrep to find offsets (ShellBat plugin creator). But he already closed an issue and tell the user to update on 3.68.
I could try to find the offset but it requires decrypted "SceShell" from 3.67. If you wish to help me, you need to provide me this file. Here is an application which allow to dump decrypted FW parts:
vitadump-master.zip
Can you please install "mDump.vpk" and launch it with kernel rights ("allow unsafe homebrew")? After about 1 minute, it should create a "ux0:dump" directory which will contain files I need: "SceShell.elf" and "SceShell_info.bin".
As those file are proprietary code, you cannot put them on github. Can you send me those files to my email address "fakemail_255 at hotmail dot fr"?
Thank you very much for your help.

I make the dump and send for you.

I received the file, thank you very much.
I will be busy next week so I take a look to dissassembled code in 10 days.
I will try to publish on this thread "testable" versions of the plugin. I will need help to validate those tests versions.

In fact, I just take a look at the code it didn't evolved between 3.65 and 3.68 (so I guess 3.67 is the same). The only think I need is the "SceShell" module NID for 3.67 and I could retreive it from "download_enabler" plugin.
So here is a version where I add this module NID to the offset test:
ShellSecBat_3.67_RC1.zip
Can you tell me if it solves the issue?
Thank you for your help.

its work for me on the 3.67, just made the download and replace the file, when restart and active h-encore, the plugin work

Thank you for the validation. I just published this version as V8 ;-)


Highlight inbound and outbond jobs on airport mouse hover

From CptVail:

Would it be possible to highlight all inbound and outbound assignments, when hovering over an airport? Maybe even in different colours?

It is really complex to implement :( Not the priority right now

Would you be happy with this solution?

Thanks for your contribution, merged into master after a small improvement.

Cool. I hadn't noticed a big hit on performance, probably because I had only selected a small number of jobs.
I hadn't considered the effect of filtering the list of jobs for every marker especially as the vast majority of markers wouldn't have any jobs.


feat: import repo page

feat: import repo page

 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.


Dynamic Resizing

This is definitely a nice-to-have, but it would be great if Tiny Care Terminal resized dynamically when you changed the size of your Terminal window!

Node Version: v8.1.2
tiny-core-terminal version: 1.3.1
Shell: bash
Terminal Program: cmd? I have no idea.
Operating System: Linux

I would love that too! :)

@mojoaxel How does it look for you. Cause for me the emojis loose some space after them. So everything is moved a bit to the right.

@rareinator I'm seeing this too.

If the window gets TOO small I also get problems with the commity box:

@mojoaxel Yeah but even when it's in full screen, it seems that the emojis aren't being displayed properly. I feel like there is not enough room on the right side. Again this is an issue with hyper. I would just love to know how you got it to work.  😀

it seems that the emojis aren't being displayed properly

If you have problems with emojis please see #92 and #44

@mojoaxel Yeah tried that but it didn't work. But if I update hyper to the newest canary release. It works just fine. So might be something the hyper team already fixed.


normalizer: memoize only with respect to a cfg

This fixes #2155. This solution was suggested by Jonathan, and seems very reasonable to me.
Simply disabling memoization within norm requests also gives correct results, but is very inefficient (particularly, for examples/calc/Long.fst which needs to normalize large calc chains and memoization plays a big role). I also tried setting all the memos in the environment to None at the time of a Cfg pop, but this misses the environments inside closures, and the recursive knots make descending into them hard.

@mtzguido this looks good to me, thanks!


Add a "custom" folder for modding cso2 client and fix sv_cheats.

#21

Thanks for the time and effort towards this.
I'm not pushing this into master yet, since I have big changes in a local branch in my machine.
It uses CMake to generate the project files (since we already need it for PolyHook 2). This should fix issues such as #14.

Thanks for the time and effort towards this.
I'm not pushing this into master yet, since I have big changes in a local branch in my machine.
It uses CMake to generate the project files (since we already need it for PolyHook 2). This should fix issues such as #14.

Nice! It would nice if you use the cmake to generate the source sdk too

Thanks for the time and effort towards this.
I'm not pushing this into master yet, since I have big changes in a local branch in my machine.
It uses CMake to generate the project files (since we already need it for PolyHook 2). This should fix issues such as #14.

Nice! It would nice if you use the cmake to generate the source sdk too

I'm looking to remove appframework, tier1, tier2 and tier3 dependencies from the launcher.
We will still have tier0.lib and vstdlib.lib as dependencies, since they have imports.
With these changes, we won't have to generate any Source SDK project files.

Nice. Do you know what the original CanCheat?
IDA's pseudocode looks like this to me:
bool CanCheat()
{
  if ( CBaseServer::m_State < 3 )
    return dword_108D2384 == 1;
  else
    return dword_10800FCC <= 1;
}

I have no clue what 108D2384 or 10800FCC are, but I think they're part of the CBaseServer class

Nice. Do you know how the original CanCheat?
IDA's pseudocode looks like this to me:
bool CanCheat()
{
  if ( CBaseServer::m_State < 3 )
    return dword_108D2384 == 1;
  else
    return dword_10800FCC <= 1;
}

I have no clue what 108D2384 or 10800FCC are, but I think they're part of the CBaseServer class. Do you know what they are?
I found other weird "backdoors" in CCSPlayer::ClientCommand.
If someone sends a specific command to the host/dedicated server, you can enable noclip. Maybe it's some GM function.
Either way, your sv_cheats fix looks good to me

the dword_10800FCC looks likes the number of players in the server(Check sub_1016DBB0).
But about the dword_108D2384, I really have no idea about it.

I think I would create a new branch before change anything next time lol.

@GEEKiDoS I got the launcher to load decrypted files, but the files have an issue.
UnCSO2 is leaving extra bytes in the end of the decrypted files due to padding.
I opened an issue there, Ochii/UnCSO2#3

I belive there's few traps while the CanCheat() returns true,
After changing the sv_cheats to 1, The game keeps dropping network packet even you are host, And you can see other's viewmodel:


:seedling: Add open ssf scorecard badge

Summary
Related issue(s)
https://github.com/open-cluster-management-io/ocm/issues/296

Fixes #

/approve
/lgtm


Align  sections name(services,testimonials,contact us) to the middle

Is your feature request related to a problem? Please describe.
The names of the section is not in the middle and hence to make it attractive I want to bring them to the middle
Describe the solution you'd like.
I would like to align them to the middle.
Describe alternatives you've considered.
N/A
Add any other context or screenshots about the feature request here.
Before:

After:

please assign this issue to me

Too easy issue


Error During NixOS Rice Installation: ms-vscode-cpptools.zip Build Failure

Encountered an error while installing rice on NixOS. The build for ms-vscode-cpptools.zip fails with the following message:
error: builder for '/nix/store/2dgl1kzzz5gadysasdlg9w9cz0azj3zh-ms-vscode-cpptools.zip.drv' failed with exit code 1;
last 7 log lines:

trying https://ms-vscode.gallery.vsassets.io/_apis/public/gallery/publisher/ms-vscode/extension/cpptools/1.22.2/assetbyname/Microsof
% Total % Received % Xferd Average Speed Time Current Dload Upload Left Speed

611931
61 63.8M
509k
0 0:03:28
0:02:08
0:01:20 717k

curl: (92) HTTP/2 stream 1 was not closed cleanly: PROTOCOL_ERROR (err 1)
error: cannot download ms-vscode-cpptools.zip from any mirror

Caused by: ms-vscode-cpptools assest server
My solution:

git checkout 6986fc39dc1371cbee27b9bf3003f0bfec74aae9

I am not sure whether the issue arise because of misconfiguration at flake.nix or vscodium.nix
PS: I don't use vscode, nvim works fine for me

Hi,
I don't think this is due to a configuration error, but rather a download issue. From what I can see, it seems that curl caused the error after something went wrong with the download. I've encountered similar curl download errors before, and usually, rebuilding solves the problem.
Could you try rebuilding again to see if the error disappears?

Understood, will try it out again.

Understood, will try it out again.

Did it work ?

Understood, will try it out again.

Did it work ?

Yup it finally works, it seems a little patience is required 😂


onivim test run

Hello there!
I tried installing the language server (in ubuntu 18.10: npm i -g purescript-language-server) and using it in onivim with no success. Onivim comes with language server support and it does seem that there's some purescript-language-server activity (in the developer console), but it doesn't do anything more than that, as it can be seen in the screenshot. I have no idea if this is oni's fault or not, I was hoping you could give me some pointers on how I could debug this, perhaps you could think of something :)

The language server is configured in oni with:
"language.purescript.languageServer.command": "purescript-language-server",
"language.purescript.languageServer.arguments": ["--stdio"],

oni support only --stdio. Can't think of anything else right now. Thanks!

For some reason I missed your answer. I'll try it out and see if it gets me anywhere. Thanks a bunch

So, as far as I can tell there's no way to print/see window/logMessage responses in onivim. I think the --config {} thing might have moved stuff further a tiny bit although it still doesn't work.
I've enabled oni debugging and this is some stuff I'm getting in the developer console, but I don't know how to proceed any further.
I've removed extra debug output that isn't related to the completion server - I hope :)
[COMPLETION] Requesting completions at line 22 and character 12
bundle.js:1 [LANGUAGE] Sending request: textDocument/completion|{"textDocument":{"uri":"file:///home/razvan/Documents/learn/purescript/src/ByExample.purs"},"position":{"line":22,"character":12}}
bundle.js:1 [SnippetCompletionProvider::getCompletions] Starting...
bundle.js:1 [Language Client] Request textDocument/completion - /home/razvan/Documents/learn/purescript/src/ByExample.purs: start
bundle.js:1 Error: Request textDocument/completion failed with message: connect ECONNREFUSED <IP_ADDRESS>:15502
    at new ResponseError (/opt/Oni/resources/app/node_modules/vscode-jsonrpc/lib/messages.js:46)
    at handleResponse (/opt/Oni/resources/app/node_modules/vscode-jsonrpc/lib/main.js:430)
    at processMessageQueue (/opt/Oni/resources/app/node_modules/vscode-jsonrpc/lib/main.js:258)
    at Immediate._onImmediate (/opt/Oni/resources/app/node_modules/vscode-jsonrpc/lib/main.js:242)
    at runCallback (timers.js:789)
    at tryOnImmediate (timers.js:751)
    at processImmediate [as _immediateCallback] (timers.js:722)
bundle.js:1 [REDUX - COMPLETION_STORE][ACTION] GET_COMPLETIONS_RESULT
bundle.js:1 [REDUX - COMPLETION_STORE][ACTION] null

The error message is definitely tied to purescript language server somehow, but I have no idea how and the the completion results... seem null or something

Closing due to lack of activity & linked repo is archived - presumably onivim 2 may not share the same issues


Support easier aliasing of latest deployment

Hi,
I've seen there's a now alias shortcut recently added, but as far as I understand it does not satisfy me, because I don't want to have a single default target in my config.
I want to be able to run something like:
now
now alias latest dev.domain.com

But I don't want to hardcode a single domain in my package.json as I can also do:
now
now alias latest prod.domain.com

I understand it may be dangerous to alias automatically without prior verifications of the deployment, but in my case for dev envs we don't care that much, and we run 2 frontend deployments in production, one being first deployed immediately to a small subset of users for real testing, and one being aliased manually after testing is done.
(as a sidenote for my own deployment usecase, it would be nice to be able to alias one target to another, like CodePush does: now alias prod-beta.domain.com prod.domain.com would make the deployment behind "prod-beta" being aliased to "prod", so at the end both domains will point to the same deployment)

now --automatic-alias - or something similar would be cool too!

In the short term, I published a really simple CLI app last night that allows you to pass in all the details: now-alias and here's another one that detects all the details automatically: now-realias.

Or a command like this:
now --alias=abc.mydomain.com

I think this is pretty common thing to do when we are doing continuous deployments.

The following might make sense as every (existing) alias has its own id:
now --alias=tqpTVSLz5jBbevVrMc1a7EvE

Thanks @TehShrike will look at that
@electerious what's the point of using that id? it looks to me more confusing than using domain from an user perspective

@slorber It would be equal to the rest of the CLI.

I'm going to work on this today!

awesome :)

Done! 🚀

thanks @leo I'm going to test (but currently have troubles don't know why I can't upgrade now locally :s)
Is there a way to link multiple aliases to the deployment at once? using multiple times the --alias option or providing a list of aliases?

@slorber Not yet. But that should be a separate issue!
If you're using Now Desktop, you shouldn't upgrade the CLI but rather wait a few minutes until it automatically updates.

Ok I'll open a separate issue
I'm not using Now Desktop but CLI, andI think it's a problem on my computer maybe related to NVM or something.
$ npm install -g<EMAIL_ADDRESS>/Users/sebastienlorber/.nvm/versions/node/v6.9.0/bin/now -> /Users/sebastienlorber/.nvm/versions/node/v6.9.0/lib/node_modules/now/build/bin/now
/Users/sebastienlorber/.nvm/versions/node/v6.9.0/lib
└──<EMAIL_ADDRESS>
$ now -v
𝚫 now 0.28.0
> UPDATE NEEDED Current: 0.28.0 – Latest 0.37.0 (released 10m ago)
> Run `npm install -g now` to update

$ which now
/Users/sebastienlorber/.nvm/versions/node/v6.9.0/bin/now

$ /Users/sebastienlorber/.nvm/versions/node/v6.9.0/bin/now -v
𝚫 now 0.37.0

@slorber I recommend removing the npm version completely and letting Now Desktop handle it. You won't have to update the CLI in the future!

Yes will give it a try thanks but it's only for mac and I'm fine with the CLI anyway :)
Was able to update finally, it was just that NVM didn't configure my path correctly and the now used was from /usr/local/bin/now

@slorber Win and Linux support coming this week!

Looks like this is not supported anymore?
I want to setup different aliases for different deployments, always wanting to alias the latest deployment, like:
now && now --alias dev.my-domain.com
OR
now && now --alias snapshot.my-domain.com

Hi! Is it still possible to do this with the Vercel CLI, or is there any alternative way?

I installed an npm package called cli-clipboard which lets you do the following:
now && now alias set $(npx clipboard) dev.my-domain.com
If you are on a Mac, you could just do:
now && now alias set $(pbpaste) dev.my-domain.com

Hidden commands are best, this is cool to still be allowed to alias something on our own

Update to above comment:
I have found a better way to do it:
now | xargs -I{} now alias {} dev.my-domain.com
if you want to alias to multiple domains:
now --force | xargs -I{} sh -c '{ now alias {} dev1.my-domain.com; now alias {} dev2.my-domain.com; }'


How can I use this within MultiMC?

The guide in the description of this fork is only for the official Minecraft launcher. I want to install this onto MultiMC since it is way easier to control different versions there and much more that is more of a personal choice. There isn't a .json file in the MultiMC launcher that's anything like the one in the official Minecraft launcher. Also, I think it would be nice to have an option of where you install the wrapper. Like, installing it in MultiMC or the official launcher or TLauncher or any launcher really. It would be pretty convenient and easy as you wouldn't have to deal with launcher specific quirks on your own and install it manually.

Hey @shadept-GitHub, I've now added a task to generate a MultiMC instance of RetroWrapper on building it, which means that all new releases of RetroWrapper now have a MultiMC instance for them! You can find a MultiMC instance for v1.6.0 in the releases tab.


Twitter scraping

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I couldn't see the twitter code for scraping here. Is there a possibility to add keyword and trend scraping?
Record

[X] I agree to follow this project's Code of Conduct
[ ] I'm a GSSOC 23 contributor
[ ] I want to work on this issue

Go ahead @4yR1l1k
Note

Create a separate module for this, as per the folder and project structure
Do not use the `selenium web driver as it is incompatible with all devices and cloud platforms.
Before making any changes, please check whether the module you want to add exists. If yes, then you can just add your functionality as a method only make a separate module and class for it.

All the best 👨💻

Any updates here?


Redesign workflow of doodba-qa

First, goals:

Scaffolding should be small as possible, because it requires most maintenance.
QA dependencies should be included, to make pipelines faster.

Next, current design:

doodba-qa includes all needed dependencies in the image.
All of them are inside the volume /qa, which gets copied and mounted in the doodba project itself before each CI job.
Those dependencies are only for the insider scripts.
Outsider scripts exist to manage images and containers (build, destroy, shutdown...), or perform some operations that require network access (closed-prs).
Insider scripts exist to execute things inside a container. Basically: install, test, coverage, lint.

Problems:

Sharing a volume is not possible or easy on all environments.
Different versions of doodba have different python versions, so it could have different C extensions. Bind-mounting the preinstalled dependencies can lead to errors.

Good ideas from https://github.com/Tecnativa/doodba-scaffolding/pull/13:

Install things using the base image, without nasty hacks.
Provide a .gitlab-ci.yml file that works out of the box.

So, what could we do to improve the situation?

[ ] Include QA dependencies in doodba itself.
This would be great because CI builds would be faster (all is included in the base image) and portable (no need to mount shared volumes).
To avoid polluting the odoo production environment, with pip version conflicts or unexpected behaviors, QA dependencies should exist in a separate virtualenv, inside the same container.

[ ] Include a tool to easily execute insider scripts in Doodba.

[ ] Dismantle all the copy & share mechanisms in Doodba QA. Not needed anymore due to previous point.

[ ] The Doodba QA image should only contain dependencies needed for Outsider scripts.

[ ] Doodba Scaffolding should include a .gitlab-ci.yml that benefits from the recently released feature (in Gitlab 11.4) of including an external file, and should get it from the example we have here. Maybe we need to change the example a bit.

[ ] Enable Gitlab CI in Doodba Scaffolding, now that we are able to use Gitlab CI with GitHub.

[ ] Enable Gitlab CI in Doodba QA. See previous point.

Benefits from this new approach

Doodba Scaffolding and QA would get CI.
CI would work out of the box.
Environment wouldn't matter.
CI would be faster.

Help wanted!
cc @PCatinean

Included dependencies in base image

The only downside I see here is having the increased docker image size both dependencies and CI scripts. While already opinionated and including some extras this would be even more to add on top. Then again if the dependencies don't add up too much it's not a big deal? (I guess).
Installing the dependencies and copying the scripts in a separate image after building the production one doesn't add much time since the layers are re-used and it keeps the production one leaner.

Dependencies in separate virtualenv

If the CI dependencies are in a different virtualenv this means we must switch between environments when doing Odoo tests and coverage and flake8?
This would again be easier with a separate image derived from production with just the purpose of testing.

Outsider scripts

If everything is bundled in the test image and this is created by a gitlab-ci job, do we still have a purpose for the outsider script? If everything is bundled during build then testing deployment and shutdown can all be handled from jobs. At least in my case that's how it is and if I want to create test instances I just use k8s to create pods with the created images and such.

The only downside I see here is having the increased docker image

Sadly Doodba is so big already that a couple of extra dependencies wouldn't add much of a problem here.

CI scripts

Actually all CI scripts would still exist in Doodba QA. Doodba itself would only include the dependencies in that separate virtualenv, and a helper to execute insider scripts.

this means we must switch between environments when doing Odoo tests and coverage and flake8?

Yes.
The purpose is to avoid disturbing normal production operation, while still havings the tools always available.

This would again be easier with a separate image derived from production with just the purpose of testing.

Easier for whom?
If you are just a dev, the easiest experience is to plug & play, everything working out of the box. Maintaining 6 extra files per scaffolding would be harder for you. Yes, it would be easier for us, Doodba maintainers, since all the CI responsibility relies on the user side, but that's not our goal.
I prefer to increase complexity in the base image and the QA project than in the scaffolding, which is what developers use most.

do we still have a purpose for the outsider script?

Yes, scripts to actually manage the deployment. It should include docker-compose for example, and let the CI pull, build, push images, run, destroy environments, etc. It would also include the insider scripts that would be executed with the executor (see checkbox number 2).
For your env, it could include k8s client for example, to be able to deploy or destroy environments.
All of those make no sense at all in the base image, if ever technically possible.

If there are two separate virtualenvironmets does that mean the CI one will be a duplicate of the server one with extra libs for the scripts?

Yes, actually the main one wouldn't be a virtualenv, and the CI one would
have access to global libs, so only the extra ones should be added.

I guess that makes more sense

I've noticed that pylint doesn't support python 3.7. In this case for V12 we need to use pylint with 3.5 and for coverage report 3.7?

AFAICS in https://pypi.org/project/pylint/, py3.7 is supported... :thinking:

It seems like that's no longer the case: https://github.com/PyCQA/pylint/commit/e17b37c5cddb620b009e91a9bc37aa333502ad74#diff-88b99bb28683bd5b7e3a204826ead112

First steps in https://github.com/Tecnativa/doodba/pull/192 and #9.


Add GCD code and Find skeleton

Make sure these boxes are checked before submitting this pull request~!~ Thank you!!

[x] The test of the method you have fixed is passing
[x] You have written a new skeleton method for someone else to work on!

In the meantime someone submitted a working PR  for this change, so I will close this one.


Document AstPass

Given that implementing QueryFragment is something that we want
plugins to be able to do, this type is part of our public API. I do not
want constructing this type to be part of our public API (it might be
required today by someone who is implementing their own connection
adapter, but we should try to make that unneccessary), so those methods
are hidden. There are also a few methods that care about the current AST
pass which I'd like to refactor out when I have the time. Those methods
are also hidden.

Relevant for anyone outside the core team who may be interested in reviewing this: I'm planning on writing a guide around extending the query builder using this code as the final goal. I'm feeling pretty good about the level of abstraction in that code (maybe not specifically for pagination, but it felt like a good self-contained example. And there will always be use cases that need to drop down to this level). However, AstPass needs to be documented for that guide (and specifically reborrow feels off to me if it's part of our public API. Unfortunately there's no good place that reborrowing for the language is documented that I can link to)


Don't run release workflow on forks

I synced my fork and noticed that it tried to run the release workflow which obviously failed. I'm not sure if this is the best way to do it or if the inherited workflow should be modified, but this works.

I took a peek at the docs at https://docs.github.com/en/actions/learn-github-actions/variables and https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows but nothing stood out to me. I'm not that well-versed in actions so I might have missed something.
I basically copied https://github.com/prisma/prisma-examples/pull/1999 which solves the same problem.


Clean out (eventually) the branch "zoom_fixes"

Complete the current plan, then when all works delete the branch zoom_fixes.

Closing for now.


Apply padding to approval icons on PR list

Rather minor nitpick, not worthy backporting.
Before:

After:

The off-center reject icon is not due to CSS; it's just how it was made.

I think if you could add it it to one of the 1.12 backports that are current it would be useful to have

ping LG-TM